From: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][Patch]: Fix context menu fontawesome icons |
Date: | 2017-09-15 14:39:44 |
Message-ID: | CAKKotZRndPSHb2LD=szuCuUjGaKiYw1mwXjZrBSexZDcNdR4JQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Fri, Sep 15, 2017 at 7:16 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>
> On Fri, Sep 15, 2017 at 2:19 PM, Murtuza Zabuawala <
> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>
>> Hi Dave,
>>
>> On Fri, Sep 15, 2017 at 6:26 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> On Fri, Sep 15, 2017 at 8:52 AM, Murtuza Zabuawala <
>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Ashesh,
>>>>
>>>> PFA minor patch to fix the font-awesome icon layout issue, attaching
>>>> screenshot of an issue for your reference.
>>>>
>>>> Regression of your commit: Link
>>>> <https://git.postgresql.org/gitweb/?p=pgadmin4.git;a=blobdiff;f=web/pgadmin/static/css/bootstrap.overrides.css;h=b6d50eb274ee1b7e05bf4b195407ca8ae9b03097;hp=f4d21b338691c69d055cfa9820f8fd1037afbd90;hb=3bbd1a8a48aac6a1f2b72dbe951f58fc32ed3bda;hpb=8b1aeadaf0347fb27bc7756cdf03aad32d03f3da>
>>>>
>>>
>>> I'm not sure this is the right way to fix this. We should re-add the
>>> left hand margin on the buttonbar menus and top-level menus. That's
>>> consistent with what the context menu looks like, but more importantly it's
>>> also consistent with (at least) macOS.
>>>
>> This is a workaround until we have proper icons for menu.
>>
>
> What do you mean? The icons were removed, and are not coming back.
>
Oh, I was not aware of that.
>
>
>>
>> If we add the margin again then Menu doesn't look good with so much
>> whitespace.
>>
>
> Maybe, but it's a standard design that we see elsewhere too - and
> consistency is good for usability.
>
That's true.
>
>
>> If you like that way then I'm attaching the patch for the same which will
>> add margins again as they were before icons.
>>
>>
>
> Thanks, applied.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
From | Date | Subject | |
---|---|---|---|
Next Message | Ashesh Vashi | 2017-09-17 19:35:22 | pgAdmin 4 commit: Moved the webcabin-docker in the 'web/pgadmin/static/ |
Previous Message | Surinder Kumar | 2017-09-15 14:19:46 | Re: v2.0-rc1 not starting, runtime not building |