From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Graphical Explain Tooltip |
Date: | 2011-07-11 14:27:44 |
Message-ID: | CA+OCxoxGrymwArBFQwWJKgXys2ijcWVTjF2Cfm0hS2Cc3Zw4tQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks, applied.
On Mon, Jul 11, 2011 at 2:09 PM, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com
> wrote:
> Hi,
>
> Current behaviour of the ExplainPopup, requires a mouse click on the
> ExplainShape to show it.
> A lot of user will not able to figure out that until
> they accidentally click on that shape or read the documents carefully.
>
> Also - current implementation does not allow to use the ExplainCanvas in a
> modal dialog.
> For some experiment, I tried to use the ExplainCanvas in a modal dialog and
> clicked on a shape.
>
> A lot of time on linux, this scenario resulted in a crash or unstable
> pgadmin application.
>
> The attached patch enables the ExplainPopup to behave like the tooltip for
> those shapes.
> As per my conversation with Dave, I have extended the ExplainPopup using
> wxTransientPopup or, wxFrame
> (where not available wxPopWindow i.e. Mac), just like the wxTipWindow, and
> also worked on the border around the
> ExplainText.
>
> I have tested it on all the platforms (i.e. OSX 10.6, Window 7 & Ubuntu
> 10.04)
>
> --
>
> Thanks & Regards,
>
> Ashesh Vashi
> EnterpriseDB INDIA: Enterprise PostgreSQL Company<http://www.enterprisedb.com>
>
>
>
> *http://www.linkedin.com/in/asheshvashi*<http://www.linkedin.com/in/asheshvashi>
>
>
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Thom Brown | 2011-07-11 14:29:51 | Re: Overhaul of type attributes modification |
Previous Message | Dave Page | 2011-07-11 14:27:14 | pgAdmin III commit: Current behaviour of the ExplainPopup, requires a m |