From: | "Dave Page" <dpage(at)pgadmin(dot)org> |
---|---|
To: | "Robins Tharakan" <tharakan(at)gmail(dot)com> |
Cc: | pgadmin-hackers(at)postgresql(dot)org |
Subject: | Re: Display line numbers in SQL view |
Date: | 2008-03-17 08:56:29 |
Message-ID: | 937d27e10803170156j46b1bb14pb422a4238b2f1d1c@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Sat, Mar 15, 2008 at 2:43 AM, Robins Tharakan <tharakan(at)gmail(dot)com> wrote:
> Hi Dave,
>
> True. In that case moving it as an option (with sane defaults) did seem like
> a better way. So attached is a reworked patch that puts two options in
> frmOptions.
>
> 1. A checkbox (in preferences tab) that applies to line numbers universally.
> 2. One checkbox (in the Query tab) specific to the SQL Query window.
>
> The defaults ensure that the we maintain status quo (i.e. Line numbers are
> enabled for Functions/Triggers but disabled for Query window).
Hmm, sorry, that still irks me. How about we just have 2 independent
options, one for dialogues, and one for windows (the query tool and
debugger)? We have various other options actually on the Query Tool
(because it can be run standalone), so the window option should
probably be included there as well.
Unless you want to incorporate this change into a wider change to move
all options onto frmOptions, and allow the appropriate parts of that
to be access from the Query Tool? That's what I'd like to see if you
want to work on it.
--
Dave Page
EnterpriseDB UK Ltd: http://www.enterprisedb.com
PostgreSQL UK 2008 Conference: http://www.postgresql.org.uk
From | Date | Subject | |
---|---|---|---|
Next Message | m.emadi | 2008-03-17 09:50:38 | Re: Translation to Persian |
Previous Message | Guillaume Lelarge | 2008-03-17 06:55:48 | Re: Translation to Persian |