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

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

In response to

Responses

Browse pgadmin-hackers by date

  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