Re: [pgAdmin4][RM2927] Move all CSS into SCSS files for consistencyand ease of colour maintenance etc.
От | Dave Page |
---|---|
Тема | Re: [pgAdmin4][RM2927] Move all CSS into SCSS files for consistencyand ease of colour maintenance etc. |
Дата | |
Msg-id | CA+OCxoxcLheG0fYoZuiUnLs8Fe3L2OiEWRKog8VyY4qfGUatOQ@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: [pgAdmin4][RM2927] Move all CSS into SCSS files for consistencyand ease of colour maintenance etc. (Aditya Toshniwal <aditya.toshniwal@enterprisedb.com>) |
Ответы |
Re: [pgAdmin4][RM2927] Move all CSS into SCSS files for consistencyand ease of colour maintenance etc.
|
Список | pgadmin-hackers |
Hi
--
On Tue, Aug 14, 2018 at 6:41 PM, Aditya Toshniwal <aditya.toshniwal@enterprisedb.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
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
В списке pgadmin-hackers по дате отправления: