Re: pgAdmin III 1.12: Toolbar issue on modern windows

From: Steffen Kuhn <pg(at)kuhnsteffen(dot)de>
To: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: pgAdmin III 1.12: Toolbar issue on modern windows
Date: 2010-09-22 18:32:26
Message-ID: 4C9A4BBA.9060808@kuhnsteffen.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Guillaume,

I tried to reproduce this this issue starting with the update line with
12.0 RC1,
but everthing worked fine from there. So not that many will be affected.
As far as I could estimate your patch fixes this issue. I need to
estimate because
of I don't get I reproduced so that I can't really verify your patch.

Regards Steffen

Am 21.09.2010 21:40, schrieb Guillaume Lelarge:
> Le 21/09/2010 15:56, Guillaume Lelarge a écrit :
>
>> Hi Steffen,
>>
>> Le 21/09/2010 12:12, Steffen Kuhn a écrit :
>>
>>> [...]
>>> today I had had the chance to check this issue on Windows 7 and everything is fine.
>>> As further check I removed the key HKCU\Software\pgAdmin III\frmStatus from the registry of my Vista box
>>> and e evoila everything is fine on Vista.
>>> I've no idea where this is coming from; may be this comes from my version beeing upgraded from the betas in combination with a missing
>>> perspective update for this case - but I just guessing.
>>>
>>>
>> We have a setting telling the size of some component
>> (FRMSTATUS_PERSPECTIVE_VER in this case). The setting depended on the
>> release number you installed. One reason could be that, since the git
>> migration, the release "number" is not updated. Previously, we used the
>> SVN commit id, which increments automatically. Maybe I forgot to do it
>> last time I changed the setting. According to GIT, it doesn't seem to be
>> happening. Probably an effect of using dev releases. Can't tell you more
>> right now.
>>
>>
> Well, I still think there is an issue here. We used the VerFromRev()
> function before when we were using svn, but its use right now make every
> parameter get the name "Perspective-" instead of "Perspective-<id>".
>
> Patch attached.
>
>
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2010-09-23 11:32:25 pgAdmin III commit: Fix a crash if pressing escape in the edit grid fil
Previous Message pgAdmin Trac 2010-09-21 20:06:38 [pgAdmin III] #237: Add a label to the "previous queries" combobox