Ant-design-pro: Ant Design Pro Preview - "dark style" / "dark theme" issues 🐛[BUG]

Created on 23 Mar 2020  ·  7Comments  ·  Source: ant-design/ant-design-pro

🐛 bug 描述

I'm not sure if this is an issue with https://github.com/ant-design/ant-design-dark-theme or with this repository, but on several places on the preview site the "dark style" / "dark theme" used does not render elements correctly.

Here's some basic things I found:

Navbar

  1. "World dropdown" wrong background & font colors:
    World dropdown navbar wrong colors

  2. "Profile Icon dropdown" wrong background & font colors:
    Profile image dropdown wrong colors

Settings / Gear Sidebar

  1. Inside styling more "dark" than the outside styling, making it look weird:
    Sidebar inside styling dark

Search Table (https://preview.pro.ant.design/list/table-list)

  1. Table head column shows wrong color on hover
  2. Table row shows wrong color (should be dark?)

Search table colors and tr highlighting

Basic Profile (https://preview.pro.ant.design/profile/basic)

  1. Table row shows wrong color (should be dark) -- same as mentioned above
    Basic profile table white

Card List (https://preview.pro.ant.design/list/card-list)

  1. Card hover disappears border of card:

hover-on-card-removes-border

Account Center (https://preview.pro.ant.design/account/center)

  1. Image icons (briefcase / sitemap / map marker) in profile aren't transparent, and so they don't blend in with background theme
  2. Tag input section colored wrong (white instead of dark)

account center images white and label:tags white

📷 复现步骤

  • Go to https://preview.pro.ant.design/
  • Click the gear on the right hand side
  • Choose "Dark style" under "Page style setting"
  • Navigate to the areas / urls I mention in the screenshots above
Inactive 🛑 bug

All 7 comments

Few other places I found:

Dashboard Monitor Page (https://preview.pro.ant.design/dashboard/monitor)

  1. Hover tooltip has white text
    hover-github-map-dashboard-monitor-page

Editor Pages (any, but example: https://preview.pro.ant.design/editor/koni)

  1. Canvas text white when should be dark (I highlighted it to show it's there)
    editor-not-overriding-styles-for-canvas

cc @chenshuai2144 , I know he's worked on a good amount of the dark theme for https://github.com/ant-design/ant-design-dark-theme

I will solve these problems step by step.
Thank you very much for your arrangement

  • [x] "World dropdown" wrong background & font colors
  • [x] "Profile Icon dropdown" wrong background & font colors
  • [x] Inside styling more "dark" than the outside styling
  • [ ] Card hover disappears border of card
  • [x] Tag input section colored wrong (white instead of dark)
  • [x] Image icons (briefcase / sitemap / map marker) in profile aren't transparent, and so they don't blend in with background theme
  • [x] Hover tooltip has white text
  • [x] Canvas text white when should be dark (I highlighted it to show it's there)

Hey @chenshuai2144,

I've been checking out the progress and have been really loving the changes! Few things extra I found based on the latest changes on https://preview.pro.ant.design/:

Mind Editor (https://preview.pro.ant.design/editor/mind)

Looks like the text just is still dark here, when it should be light:

Screen Shot 2020-04-07 at 5 40 29 PM

Setting panel - Inside styling more "dark" than the outside styling (clarification)

This one I mentioned before, but just to make sure I'm clear, I tried to point out exactly what I'm talking about:

Screen Shot 2020-04-07 at 5 41 39 PM

Basically, there's a padding of a lighter black around the darker black in the Settings side panel, and it looks kind of odd, especially when near text, and near things like the "x" in the top right corner.

Settings panel - "Dark style" mentioned twice

Quick gif to explain:

dark-theme-twice

In the settings sidebar, the right two themes are both called "dark style", even though really the differentiation should maybe be "dark sidebar" and "dark".

Settings panel - clicking theme changes url, but does not persist on reload:

Another quick gif to explain:
dark-theme-not saving-on-refresh

Basically, if you load the page in the normal theme, and change the theme to something like "dark style", the url changes to:

https://preview.pro.ant.design/dashboard/analysis?navTheme=realDark&primaryColor=daybreak

I'd expect the navTheme=realDark&primaryColor=daybreak params to mean that the theme would load and persist when refreshing that url, but that doesn't seem to be the case.

Overall

Just want to say I really appreciate you listening to my feedback! Hope you are having a great week ❤️

Providing useful suggestions is very helpful for open source projects
❤️

@chenshuai2144 Possible new regressions?

Screen Shot 2020-06-08 at 9 25 14 AM

new version will change this style

Was this page helpful?
0 / 5 - 0 ratings

Related issues

zhongjiewu picture zhongjiewu  ·  3Comments

lvzheng0404 picture lvzheng0404  ·  3Comments

renyi818 picture renyi818  ·  3Comments

RichardStark picture RichardStark  ·  3Comments

happier2 picture happier2  ·  3Comments