Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UI issues with the v1 dark theme (pre-redesign) #2988

Closed
31 of 41 tasks
turt2live opened this issue Jan 19, 2017 · 39 comments
Closed
31 of 41 tasks

UI issues with the v1 dark theme (pre-redesign) #2988

turt2live opened this issue Jan 19, 2017 · 39 comments
Labels

Comments

@turt2live
Copy link
Member

turt2live commented Jan 19, 2017

"Issues" being a loose term. This is intended to be a collection of hard-to-see and similar oddities in the dark theme.

Edit: Reference #2984
Edit: I'll try to keep this list updated as issues are fixed/added


@alexmat2on
Copy link

alexmat2on commented Jan 19, 2017

In the following screenshot, there are two issues:

  • The currently opened room's background color is pure black, which looks out of place with the slate/dark grey of everything else. The selected room should be darker than the others, but not that dark. In the screenshot, I have #riot open.

  • When you hover over other rooms, the room icon gets an overlay to indicate it's being hovered on. The overlay seems to be too light though, (i.e., it lightens the room icon) which also feels out of place in the dark theme. Perhaps darkening would work better. In the screenshot, I'm hovering over #matrix.

dark theme issues

@turt2live
Copy link
Member Author

turt2live commented Jan 19, 2017

  • Another case: Ping notifications are a bit harsh on the eyes.
    image

(I quite like it like this... --Matthew)
~ Fair point. It does entice me to actually read my messages. - Travis

@alexmat2on
Copy link

alexmat2on commented Jan 19, 2017

  • The light-grey text on white background in these sorts of menus is a little hard to read:

  • 2light1

  • 2light2

@turt2live
Copy link
Member Author

turt2live commented Jan 19, 2017

  • Start chat dialog (Chrome).

image

@turt2live
Copy link
Member Author

turt2live commented Jan 19, 2017

  • Mute icon under the room's context menu for notifications
    image

@ara4n
Copy link
Member

ara4n commented Jan 19, 2017

  • dark icons in general are too dark - e.g. padlock, close-buttons, photo button on User Settings.

thanks for enumerating all these (and continuing to do so); will get them all sorted before we release it.

For big lists like this, please do use * [ ] markdown notation to make it easy to track all the subissues and check them off as they're sorted.

@ara4n ara4n mentioned this issue Jan 20, 2017
@ara4n
Copy link
Member

ara4n commented Jan 20, 2017

  • non-Gemini scrollbars are too light:

screen shot 2017-01-20 at 20 11 00

  • ...and gemini ones are too dark

@ara4n
Copy link
Member

ara4n commented Jan 31, 2017

  • selector when inviting is white-on-white

@turt2live
Copy link
Member Author

turt2live commented Feb 2, 2017

  • Manage integrations dialog - close button, text, icons
    image

More screenshots for integrations:

  • Github (adding)
    image

@turt2live
Copy link
Member Author

turt2live commented Feb 3, 2017

  • RTE
    image

Steps:

  1. Enabled RTE
  2. Clicked bold (without text in field)

Entering text after the asterisks causes it to be readable, however backspacing back to the asterisks causes the same effect as above.

@turt2live
Copy link
Member Author

turt2live commented Feb 3, 2017

  • Logging out of an account that had the dark theme enabled. (This is post-logout, no page refresh)
    image

Edit: Added more information

  • And then logging in to an account later (guest, in this case)...
    image

@turt2live
Copy link
Member Author

turt2live commented Feb 3, 2017

  • Some images end up caching to the light version for long-time light theme users (or those starved of having the dark theme until recently).
    image

This is probably a non-issue in a way.

@turt2live
Copy link
Member Author

turt2live commented Feb 7, 2017

  • Join room 'room does not exist' message
    image

@turt2live
Copy link
Member Author

turt2live commented Feb 8, 2017

@lukebarnard1
Copy link
Contributor

mx_Login_field_prefix, mx_Login_field_suffix background colour

2017-05-08-161844_530x583_scrot

@turt2live
Copy link
Member Author

For those not watching this aggressively, I've updated the OP.

@t3chguy
Copy link
Member

t3chguy commented Jun 9, 2017

I think "Unencrypted room" indicator is done now. It was an ambiguous inconsistency, one of my first PRs :P
image

@tessgadwa
Copy link

@t3chguy are you still working on this theme?

@t3chguy
Copy link
Member

t3chguy commented Jun 9, 2017

Me? Working on this theme? I fixed any issues that hurt my eyes is all :P
I have 0 UI skillz

@tessgadwa
Copy link

Cool. I've been wanting to dive back in, but a little discouraged since my PR for the light contrast theme never completed review.

@ara4n
Copy link
Member

ara4n commented Jun 9, 2017

the light contrast theme is still on my radar; we are just drowning in PRs and releases atm. I haven't forgotten it.

@turt2live
Copy link
Member Author

@t3chguy there's two icons for the encrypted icon, the one you mention is fixed:
image

@t3chguy
Copy link
Member

t3chguy commented Jun 9, 2017

ah @turt2live I must've missed it
that one is plainly missing an invert, I will sort it

@t3chguy
Copy link
Member

t3chguy commented Jun 9, 2017

@turt2live sorted :)

@tessgadwa
Copy link

thanks @ara4n all good.

@turt2live
Copy link
Member Author

Encrypted room icon in room settings is purple now...
image

@t3chguy
Copy link
Member

t3chguy commented Jun 13, 2017

oh heck, I guess it should only have the flipColour class when its unlocked. I shall fixy @turt2live
...
fixed by #1088

@uhoreg
Copy link
Member

uhoreg commented Jul 5, 2017

@grahamperrin
Copy link

Texts: legibility of dark blue and other troublesome colours on the dark grey background

Borrowed from issue 4403:

2017-07-30 11 03 35 without mx_roomheader_avatar

Troublesome colours include:

  • dark blue (e.g. the word Shrustika)
  • mid blue (e.g. geraldobarros)
  • red (e.g. michaelkohler)
  • purple (e.g. karanganesan and Jandit).

Related

Optionally include higher contrast colour scheme · Issue #2300 · vector-im/riot-web

@Mikaela
Copy link
Contributor

Mikaela commented Jul 31, 2017

@grahamperrin I think that is an issue with the bot using those colours rather than Riot as I think it would also affect many IRC clients with dark backgrounds.

@grahamperrin
Copy link

With a different bot, a different troublesome colour (darker purple, I think):

2017-08-02 18 58 45

Should there be separate issues for each bot?

@t3chguy
Copy link
Member

t3chguy commented Aug 2, 2017

@grahamperrin the bot related issues should probably end up in their respective repository

@aidalgol
Copy link
Contributor

The mx_filterFlipColor apprach seems to me to put the logic in the wrong place. Why not instead select these same elements, by their semantic names, in the SCSS code, instead of making styling decisions in the components?

@grahamperrin
Copy link

grahamperrin commented Jan 26, 2019

label: redesign, https://github.com/vector-im/riot-web/labels/redesign

… or, is the dark theme in the current redesign distinct from this older new dark theme?

@turt2live
Copy link
Member Author

The new new dark theme stuff is being tracked here: #8554

@turt2live turt2live changed the title UI issues with the new dark theme UI issues with the v1 dark theme (pre-redesign) Feb 13, 2019
@aaronraimist
Copy link
Collaborator

I don't see anything else in here that is still applicable to the new dark theme

@ghost
Copy link

ghost commented Jun 9, 2020

Close this, and replace #8554 on your Roadmap ->

SMALL: Implement Black Theme and make dark theme lighter [issue]

I actually believe adding this to First Time User Experience roadmap would be better. The current Dark Theme doesn't look good. Example; the lines
image
image

And the loading, as said before. But the outlines around the loading box doesn't have the harsh lines which should be default everywhere 👍
image

@yannicka
Copy link

  1. UI issues with the v1 dark theme (pre-redesign) #2988 (comment) seems to be fixed
  2. Checkboxes "Autocomplete field not themed" and "Room directory join icon too dark join button broken on dark theme roomdirectory #3349" can be checked

@turt2live
Copy link
Member Author

I'm closing this as it is now 2 brands old and isn't really in a useful format by today's standards - please open new issues for each issue, though preferably they are zoned for the design team to take a look when they encounter the area.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests