Thread: [pgAdmin4][Patch]: Fix context menu fontawesome icons

[pgAdmin4][Patch]: Fix context menu fontawesome icons

From
Murtuza Zabuawala
Date:
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

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

https://community.postgresrocks.net/
Attachment

Re: [pgAdmin4][Patch]: Fix context menu fontawesome icons

From
Dave Page
Date:
Hi

On Fri, Sep 15, 2017 at 8:52 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.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

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. 

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Re: [pgAdmin4][Patch]: Fix context menu fontawesome icons

From
Murtuza Zabuawala
Date:
Hi Dave,

On Fri, Sep 15, 2017 at 6:26 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Fri, Sep 15, 2017 at 8:52 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.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

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. 

If we add the margin again then Menu doesn't look good with so much whitespace.
If you like that way then I'm attaching the patch for the same which will add margins again as they were before icons. ​
 

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Attachment

Re: [pgAdmin4][Patch]: Fix context menu fontawesome icons

From
Dave Page
Date:


On Fri, Sep 15, 2017 at 2:19 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi Dave,

On Fri, Sep 15, 2017 at 6:26 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Fri, Sep 15, 2017 at 8:52 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.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

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.
 

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.
 
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

Re: [pgAdmin4][Patch]: Fix context menu fontawesome icons

From
Murtuza Zabuawala
Date:
On Fri, Sep 15, 2017 at 7:16 PM, Dave Page <dpage@pgadmin.org> wrote:


On Fri, Sep 15, 2017 at 2:19 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi Dave,

On Fri, Sep 15, 2017 at 6:26 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Fri, Sep 15, 2017 at 8:52 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.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

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