Hi Dave,
On Fri, Sep 15, 2017 at 6:26 PM, Dave Page <dp...@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
>> <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.
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
>
diff --git a/web/pgadmin/static/css/bootstrap.overrides.css
b/web/pgadmin/static/css/bootstrap.overrides.css
index 94fb449..82c2f4e 100755
--- a/web/pgadmin/static/css/bootstrap.overrides.css
+++ b/web/pgadmin/static/css/bootstrap.overrides.css
@@ -105,7 +105,7 @@ iframe {
border-width: 5px 0 5px 5px;
border-left-color: #cccccc;
margin-top: 5px;
- margin-right: -2px;
+ margin-right: -10px;
}
.dropdown-submenu:hover>a:after {
@@ -153,10 +153,6 @@ iframe {
margin-bottom: 0;
}
-.dropdown-menu > li > a {
- padding: 3px 5px;
-}
-
.dropdown-menu > li > a > i {
margin-left: -17px;
width: 13px;
