Re: pgAdmin III: Muliple SQL tabs

From: Sergey Busel <sbusel(at)gmail(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: pgAdmin III: Muliple SQL tabs
Date: 2016-02-03 03:07:09
Message-ID: CAK-Xf3tAXfqpptEcNdwMuM41Gs=7QZjd5fnf1Efv4itweSce0A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

- Removed unused/commented code.
- Tab names now reflect the file name, if such is associated with a tab.
- Removed the "Close Tab" menu item. Added X button to the active tab.
- To tell the user which tab results are related to and to avoid messing
with perspective, the name of the related tab is now displayed as "Data
Output [Query 1]" in the output pane. If the tab is named after a file,
file name will display in the square brackets. If the file name is longer
then 15 chars, it will be truncated to 15 chars.

Let me know any other suggestions you may have.

Thanks.

On Tue, Feb 2, 2016 at 5:08 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

> On Tue, Feb 2, 2016 at 3:17 AM, Sergey Busel <sbusel(at)gmail(dot)com> wrote:
> > Patch file for multiple SQL tabs is attached.
>
> Thanks - I see how this could be useful. I think it needs a little
> work before it could be included:
>
> - Unused code should be removed, not commented out.
> - I think the tab names should reflect the file name (where there is
> one) - e.g. "Query 2" should become "foo.sql" (without the path) if
> you save the query.
> - There should be some way for the user to see which tab the results
> are related to. Maybe the output pane title should read "Output Pane
> (Query 2)" or "Output Pane (foo.sql)" as appropriate? I'm not sure if
> that will do weird things with the perspective though - they used to
> include panel titles.
> - Instead of the "Close Tab" menu option, perhaps an X button should
> be added to the right-hand end of the active tab?
>
> Thanks.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

Attachment Content-Type Size
sqltabs.patch text/x-patch 33.6 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Sergey Busel 2016-02-03 03:25:53 Re: pgAdmin III: Muliple SQL tabs
Previous Message John Obaterspok 2016-02-02 18:45:42 Re: pgAdmin III: Muliple SQL tabs