From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][RM2927] Move all CSS into SCSS files for consistency and ease of colour maintenance etc. |
Date: | 2018-08-15 12:17:40 |
Message-ID: | CA+OCxoxcLheG0fYoZuiUnLs8Fe3L2OiEWRKog8VyY4qfGUatOQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi
On Tue, Aug 14, 2018 at 6:41 PM, Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
> Hi Hackers,
>
> Attached is the updated patch fixing the errors. I missed the linter in
> webpack.config.js :(
>
That's better - thanks.
A few thoughts:
- The button bars now have the flat style, however disabled buttons are
almost indistinguishable from enabled ones. If we're going to have a
flatter style, then we need to make the foreground colour of the buttons
much lighter when they're disabled.
- I'm not keen on using $color-white/$color-black. That pre-supposes what
the colours are, in which case we might as well just hard-code the values.
How about $color-foreground/$color-background?
- One thing we may want to do in the future is to have the option for
alternate themes (or at least just a dark one). Do you know (no need to do
it now - I'm just asking) if we've got enough of the colours in
_default_variables.scss (and overridden elsewhere) to be able to do that?
> I have noticed one more thing, templated CSS. We should remove it. For now
> I have skipped template css from webpack bundling.
>
+1
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Xuri Gong | 2018-08-15 15:56:45 | Re: [pgAdmin4][patch]: Feature #1407 - Support map view for PostGIS query result sets |
Previous Message | Aditya Toshniwal | 2018-08-14 17:41:14 | Re: [pgAdmin4][RM2927] Move all CSS into SCSS files for consistency and ease of colour maintenance etc. |