Element-web: Dark theme follow up issues

Created on 13 Feb 2019  路  35Comments  路  Source: vector-im/element-web

bug p1 minor pills theming

Most helpful comment

The spinner looks particularly bad on the dark theme I think. It's got a white line around the edges etc.

image

All 35 comments

Awesome! Small stuff, but:

  • [x] E2E icons should use primary/notice colours (fixed in https://github.com/matrix-org/matrix-react-sdk/commit/feb92957c0aca8782455a8eff83cae885613e87a)
  • [x] Vertical bar splitting top right icons isn't skinned (should we take the primary icon colour and apply 50% opacity?) (fixed in https://github.com/matrix-org/matrix-react-sdk/commit/4409636c0b969d12785eaee4e3d127ebcfdf4381)
  • [x] (+) icons in room list not skinned
  • [x] @mention backgrounds not skinned (for now using the room highlight bg would be ok) (fixed in https://github.com/matrix-org/matrix-react-sdk/commit/422750082d16017599625df04637bfb1f4050c15)
  • [x] Adjust gradient to use rgba for Safari (fixed in https://github.com/matrix-org/matrix-react-sdk/commit/5f6907c85eafd5083e000bf4b977a492544e547c)
  • [x] Change box shadow on top left menu to use bg value (fixed in https://github.com/matrix-org/matrix-react-sdk/commit/f64f39f773cbe484d2dbeb1c1d2cb78d607f7574)
  • [x] LLP should be #15171B not #1C1F24 (subtle, but important to establish the same depth hierarchy as the light theme)
  • [ ] E2E composer decoration, and message hover decoration should use new palette (we're missing a yellow, maybe use the orange if we don't have a yellow when you tackle this)
  • [x] Tweak color values to v2 from Nad (fixed in https://github.com/matrix-org/matrix-react-sdk/commit/e09963cdfb660019377f9046ddbe151e0e6949e5)
  • [x] Markdown button status appears reversed
    image
  • [x] Should there be a more visible vertical line here?
    image
    image
  • [ ] It would be nice to use a white [m] logo in the room directory
    image
  • [x] "INVITED" in user list is hard to see (https://github.com/matrix-org/matrix-react-sdk/pull/2656)
    screen shot 2019-02-13 at 7 47 37 am
  • [x] Room settings that you can't update are hard to read
    screen shot 2019-02-13 at 7 48 46 am
  • [ ] Safari scrollbars are dark but (one of the) Firefox scrollbars is light. Which is intended? Is Gemini getting killed before 1.0?

Can I chuck in:

  • [x] \ image
  • [x] [dialogs don't feel like they have enough of a backdrop on the dark theme 路 Issue #8584](https://github.com/vector-im/riot-web/issues/8584)
  • [x] [no hover effect on messages in new dark theme](https://github.com/vector-im/riot-web/issues/8580) (fixed in https://github.com/matrix-org/matrix-react-sdk/pull/2654)
  • [x] [Pills are hard to read in new dark theme 路 Issue #8587](https://github.com/vector-im/riot-web/issues/8587)
  • [x] Widgets blend in with the timeline
    image

@jryans re widgets blending; they should be using the header/panel background colour

(could people put names next to their issues if possible so I don't have to mouse over all of them?)

The spinner looks particularly bad on the dark theme I think. It's got a white line around the edges etc.

image

  • [x] Also, the redacted message bar thing seems much more stark than on the light theme. (fixed in matrix-org/matrix-react-sdk@91c91c5ca9cec166d90b71211a4f14b61ffcdf93)

image

re: redaction bar. There is a dark theme asset from the old design somewhere.
image

  • [x] [make dark names more readable when using dark theme 路 Issue #8611](https://github.com/vector-im/riot-web/issues/8611)
  • [x] [Markdown indicator on dark theme is invisible when in RTE mode 路 Issue #8637](https://github.com/vector-im/riot-web/issues/8637)
  • [x] 'back' Chevron in member panel

screenshot 2019-02-14 at 18 31 30

  • [x] [Redaction thingy](https://github.com/vector-im/riot-web/issues/8559)
  • [ ] [File drop UI](https://github.com/vector-im/riot-web/issues/8663)
  • [ ] [Concerns with color scheme being hard on the eyes](https://github.com/vector-im/riot-web/issues/8664)

@Cadair https://github.com/vector-im/riot-web/issues/8554#issuecomment-463222141 :)

  • [x] ["INVITED" header in member list](https://github.com/vector-im/riot-web/issues/8679)
  • [x] [Member search is impossible to read on dark theme and has no clear button](https://github.com/vector-im/riot-web/issues/8673)

Dark theme is very contrast - it is heavy for eyes.
Old dark theme was more nice for eyes. Do you can add third theme with old grey color?
1

  • [ ] [visibility labels on roomdirectory need to be designed and inverted on dark theme](https://github.com/vector-im/riot-web/issues/8721)
  • [ ] [Distinguishing rooms with new events is hard in the dark theme](https://github.com/vector-im/riot-web/issues/8729)
  • [ ] [Code Scrollbars in dark theme are invisible](https://github.com/vector-im/riot-web/issues/9052)

Spinners should _definitely_ be inverted to match the background. I don't know if this is OS聽X-specific, but on here it looks awful.

The spinner on a dark background

Also, the desktop app's window title bar on OS X is still light, even when using the dark theme. I'm not quite sure whether to put that here or somewhere else, since it's desktop-specific, but it's definitely noticeable, pretty much forcing me to use the client full screen. (The latest OS X has system-wide dark mode, and apps have to explicitly opt in to get dark-themed.)

Username highlights on the dark theme look awful while unselected due to the brownish background colour:

unselected

It's a bit better when selected since it then changes to a grey shade:

selected

Also, the desktop app's window title bar on OS X is still light

This is also the case with Windows

When inactive yes, when active it matches the theme colour as expected.

image

@t3chguy, huh, that's strange. Here's a screenshot with the window active:
Annotation 2020-03-09 192258

I've never seen the title bar styled properly.

riot-web version: 1.5.12
olm version: 3.1.3

image

What does your explorer window look like in comparison?

Dark - I have windows dark theme on. I have set 'follow system theme set' in riot web settings

image

I have the same issue as @Mattwmaster58 on windows 10 with dark mode enabled.

Was this page helpful?
0 / 5 - 0 ratings