From: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
Cc: | pgadmin-hackers(at)postgresql(dot)org, Dave Page <dpage(at)pgadmin(dot)org> |
Subject: | Re: PATCH(WIP): Printing Support And Save GQB/Explain as an image |
Date: | 2009-12-07 09:09:52 |
Message-ID: | 8cf965d30912070109n446f1662w9622a14412d7a540@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Guillaume,
On Mon, Dec 7, 2009 at 1:29 PM, Guillaume Lelarge <guillaume(at)lelarge(dot)info>wrote:
> It seems I keep forgetting to answer you on this mail.
>
> Le samedi 21 novembre 2009 à 19:51:07, Ashesh Vashi a écrit :
> > [...]
> > > Two comments:
> > >
> > > * the Explain menu for print preview/print/save as image should be
> > > disabled if
> > > the explain is not already available (so that the user doesn't get a
> > > warning
> > > message telling them they couldn't have the print of the explain).
> > >
> > > * I don't like the location and the text of the item "SaveAs image". I
> > > think
> > > it would be better to label it "Save As Image" (note the space between
> > > Save and As). Its location should be just under the "Save As" item,
> > > probably mixed
> > > with it (something like:
> > >
> > > Save As -> Query (text)
> > > Explain (image)
> > > GQB (image)
> >
> > Sure, I will send ths updated patch soon.
> >
>
> Do you have an updated patch available somewhere?
>
Not yet. :(
>
> > > Anyways, nice work :)
> >
> > Thanks.
> >
> > Facing couple of issues, as described below:
> > 1. On OSX (Mac):
> > - If I see a print preview for any of Query/Explain/GQB, it works
> > perfectly.
> > But, when we close the Print-Preview dialog, the query dialog is not
> > getting enabled.
> > Something is missing, but I am not able to figure out.
> > 2. On all the platforms:
> > - If I call the new function updateModelSize(...), when we add new
> table
> > to the model.
> > - In this case, after selecting a table/view from the left size tree
> > (browser), if I click couple of times (five-six times frequently) , then
> > pgAdmin3 is crashing.
> > - I tried to use the mutex/Semaphore, but it was still not working. :(
> > (For the same reason, I have not added the updateModelSize(...)
> > function, while creating/deleting a table/view, in this patch)
> > 3. Resolution problem in GQB/Explain (Print/Print-Preview). It does not
> > look good. The text is fading a little bit.)
> >
> > Would you please help me on this?
> >
>
> Or you're probably waiting on me anwering these questions before working on
> the new patch.
>
> I can help you on these tonight. I should stop deffering this work. Are
> your
> questions still current?
>
I was waiting for your answers on these questions, or anybody from the
pgadmin-hackers group.
I have something else on my plate for right now. So, I am not able to spend
time on this now-a-days. :(
--
Thanks & Regards,
Ashesh Vashi
EnterpriseDB INDIA - The Enterprise Postgres
Company<http://www.enterprisedb.com/>
"Make everything as simple as possible, but not simpler..." -- Albert
Einstein
"We are what our thoughts have made us; so take care about what you think."
-- Swami Vivekananda
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2009-12-07 09:21:04 | Re: PATCH(WIP): Printing Support And Save GQB/Explain as an image |
Previous Message | Guillaume Lelarge | 2009-12-07 07:59:29 | Re: PATCH(WIP): Printing Support And Save GQB/Explain as an image |