Re: ReadOnly ComboBoxes in dlgSelectConnection.cpp

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: pg(at)kuhnsteffen(dot)de
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: ReadOnly ComboBoxes in dlgSelectConnection.cpp
Date: 2010-08-23 08:39:25
Message-ID: 4C7233BD.6020908@lelarge.info
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

Le 22/08/2010 11:48, Steffen Kuhn a écrit :
> [...]
> if pgAdmin is startet with option 'S' and there are no QuickConnect
> informations in the settings
> the readonly comboboxes make no sense.
>

Yes, this was a known bug (at least to me), but didn't find the time to
work on it.

> I tried following code:
> if (form == NULL)
> {
> cbServer->SetValue(settings->Read(wxT("QuickConnect/server"),
> wxEmptyString));
> if (cbServer->IsEmpty())
> cbServer->SetWindowStyleFlag(cbServer->GetWindowStyleFlag()
> & ~wxCB_READONLY);
>
> cbDatabase->SetValue(settings->Read(wxT("QuickConnect/database"),
> wxEmptyString));
> if (cbDatabase->IsEmpty())
>
> cbDatabase->SetWindowStyleFlag(cbDatabase->GetWindowStyleFlag() &
> ~wxCB_READONLY);
>
> cbUsername->SetValue(settings->Read(wxT("QuickConnect/username"),
> wxEmptyString));
> }
>
> but the combobox didn't change to be editable.

The wxwxidgets doc says a refresh is needed. I did it, but it still
didn't work.

> So find another patch attached.
> Also fixed height of username combobox.
>

I did a few changes to your patch. Here is the one I commited on 1.12.
See patch attached.

I commited a slightly modified fix for 1.13 to take care of the new
rolename combobox.

Thanks for your report and patch.

--
Guillaume
http://www.postgresql.fr
http://dalibo.com

Attachment Content-Type Size
dlgSelectConnection.patch text/x-diff 3.2 KB

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message pgAdmin Trac 2010-08-23 08:47:07 [pgAdmin III] #228: Higher time resolution for pgAdmin jobs
Previous Message Guillaume Lelarge 2010-08-23 08:35:41 pgAdmin III commit: Make the dlgSelectConnection combobox read/write