Re: PATCH(WIP): Printing Support And Save GQB/Explain as an image

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

In response to

Responses

Browse pgadmin-hackers by date

  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