Re: Request for UI freeze break

Matthias Clasen <[email protected]>
Newsgroups gmane.comp.gnome.release-team,gmane.comp.gnome.documentation
Message-ID <CAFwd_vBONVOui89eTzHwjvr3MCcOfDpqihN=F7b5rNUkHFF+EQ@mail.gmail.com>
On Thu, Sep 4, 2014 at 11:18 AM, Andre Klapper <[email protected]> wrote:
> On Thu, 2014-09-04 at 10:41 -0400, Ondrej Holy wrote:
>> I'd like to request UI freeze break for the Gnome Control Center to fix wrong padding in the User accounts panel. Patches are attached in the Bug 736009:
>> https://bugzilla.gnome.org/show_bug.cgi?id=736009
>>
>> The UI change is shown in a screenshot:
>> https://bug736009.bugzilla-attachments.gnome.org/attachment.cgi?id=285311
>
> Looks good to me (r-t approval 1 of 2).
>

Second approval for the release team.

The patches are not quite perfect, since you only query the style
context once for the padding values when the page is setup. For extra
credit, you should redo that when the theme changes. But since you got
the a-c-n in bugzilla already...
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.