public inbox for [email protected]
help / color / mirror / Atom feedFrom: Björn Harrtell <[email protected]>
To: Dave Page <[email protected]>
Cc: Guillaume Lelarge <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: ]GTK 3
Date: Thu, 7 Jan 2016 12:56:38 +0100
Message-ID: <CANhDX=aixcuBpTy0-Ciu-cPfKYJ_uZQa7ZtAK0NfnXUnR-3ZiA@mail.gmail.com> (raw)
In-Reply-To: <CA+OCxoxERDMQq_0AN9-B=eGTV=T+B=JvjkNzJgaRttOvxi4SBg@mail.gmail.com>
References: <CANhDX=a8ZJUvWTDbx7eCc2bqe9i6xkZKTcRyPXpb6p3uEGaWyg@mail.gmail.com>
<CAAeC-SsGLG7_9mAsZpCZjX3TrQr_C+su3erM+vvsi=KkfatfKg@mail.gmail.com>
<CANhDX=b91i+xq1z9YWyXvGAhfHAD4gwG1QOAx3sJ7xv+p0PijQ@mail.gmail.com>
<CANhDX=bE9Wx6D2jNzDcFrMGvMiRTHfW9Svup6SAxzV2Pb=BdvQ@mail.gmail.com>
<CAECtzeXhbbCG7XPgeRpMKrnpRxJO6HsnE9v=vgxEb+ndFps7AQ@mail.gmail.com>
<CA+OCxowUQ=B3kCyKfOFiiMSkLhj+oz5os=fs=0ihEqvv4r+UZw@mail.gmail.com>
<CANhDX=YhqQtmDRo8Gb0Wrmg05bGwX690zpZxH4-F-DbCvbrXog@mail.gmail.com>
<CA+OCxoxERDMQq_0AN9-B=eGTV=T+B=JvjkNzJgaRttOvxi4SBg@mail.gmail.com>
List-Unsubscribe: <mailto:[email protected]?body=unsub%20pgadmin-hackers>
2016-01-07 12:10 GMT+01:00 Dave Page <[email protected]>:
> On Thu, Jan 7, 2016 at 7:47 AM, Björn Harrtell <[email protected]>
> wrote:
> > Yes, sorry, I made both the comments and the patch too hastily and agree
> > that it should not be removed based on my findings.
> >
> > It should be noted though that wx3 or GTK3 by themselves is not to blame
> > here, I'm guessing it should affect wx2 and GTK2 too because it's more
> of a
> > logical problem in where a user created the server node in a light theme
> > which will save a white color for the background... then if the user
> > switches to a dark theme, the saved white color is problematic.
>
> Well the same applies in a more default theme if you use a dark
> background. I guess the answer is to say "don't do that then" - I'd
> certainly prefer not to have pgAdmin try to second guess what colours
> might work against whatever the theme has for the current text colour.
>
Agreed. But what about saving the color only when making an active choice
so that the current system window background color is used until the user
have customized the color?
>
> > I noticed work on pgadmin4 too and while interesting, I want to see
> pgadmin3
> > shine on linux desktops in the life it has left which might be quite some
> > time (?). I see several distinct areas of work that I'd like to
> contribute
> > efforts with:
> >
> > 1. Upgrade to latest stable wx
>
> That should be largely done, at least as far as compilation/linking is
> concerned. As far as I know the only outstanding issue is with
> resource generation when creating a Mac package.
>
Great. Just upgrading will be beneficial. Looking forward especially to get
an updated wxStyledTextCtrl for the query editor. Unfortunately I cannot
help with Mac-issues.
>
> > 2. Compile with GTK+ 3.x as default on Linux (not sure it's ready for
> this,
> > needs research and collaboration with dist packaging)
>
> OK.
>
> > 3. Evaluate which controls that can/need replacement (i.e replace with
> > wxTreeListCtrl, wxDataViewCtrl and perhaps wxToolBar)
>
> I think we'd want compelling reasons to use different controls with
> wx3 - and they'd need to be compelling enough to warrant additional
> code complexity that would be added to continue to support wx2.
Agreed. If the changes required are very large and/or invasive it is
probably not worth it, that's what I'm planning to look into. The main
benefit as I see it is that the new controls are backed by native controls
for the GTK and Mac versions of wx.
>
> > 4. Refactor existing code to use the controls that pass evaulation
> >
> > Feedback and guidance on how to proceed is much appreciated!
>
> Propose changes here, and justify why they're needed essentially. If
> consensus is that a change is warranted, then work can start on a
> patch.
>
> Please do bear in mind though that I have something like 10 people
> working almost exclusively on pgAdmin 4 right now, pushing us to an
> alpha state by the end of Q1 - and whilst I expect pgAdmin 3 to be
> around for a while yet, my, and my teams focus will be primarily on
> that work.
>
Understandable. I look forward to be able to try out that alpha, and to
hopefully contribute to that project instead in the future.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
view thread (12+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected]
Subject: Re: ]GTK 3
In-Reply-To: <CANhDX=aixcuBpTy0-Ciu-cPfKYJ_uZQa7ZtAK0NfnXUnR-3ZiA@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox