From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
---|---|
To: | pgadmin-hackers(at)postgresql(dot)org |
Cc: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>, Dave Page <dpage(at)pgadmin(dot)org> |
Subject: | Re: PATCH(WIP): Printing Support And Save GQB/Explain as an image |
Date: | 2009-12-07 07:59:29 |
Message-ID: | 200912070859.29288.guillaume@lelarge.info |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
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?
> > 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?
--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com
From | Date | Subject | |
---|---|---|---|
Next Message | Ashesh Vashi | 2009-12-07 09:09:52 | Re: PATCH(WIP): Printing Support And Save GQB/Explain as an image |
Previous Message | Guillaume Lelarge | 2009-12-05 23:00:51 | Use of wxColourPickerCtrl |