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

Date separator lines are barely visible in light mode #22122

Closed
robintown opened this issue May 9, 2022 · 5 comments · Fixed by matrix-org/matrix-react-sdk#12343
Closed

Date separator lines are barely visible in light mode #22122

robintown opened this issue May 9, 2022 · 5 comments · Fixed by matrix-org/matrix-react-sdk#12343
Assignees
Labels
A-Timeline O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Design Z-Confidence-High High confidence in the enhancement or suggestion based on known factors, or as described.

Comments

@robintown
Copy link
Member

Steps to reproduce

  1. Use light mode
  2. Look at a date separator in the timeline

Outcome

What did you expect?

The date separator line should have a decent level of contrast

What happened instead?

They're barely visible:

Screenshot 2022-05-09 at 11-50-00 Element 1 Test room

For comparison, they have significantly more contrast in dark theme:

Screenshot 2022-05-09 at 11-50-59 Element 1 Test room

Operating system

NixOS unstable

Browser information

Firefox 99.0.1

URL for webapp

develop.element.io

Application version

Element version: 1783645-react-e05a3e644659-js-706b4d605481 Olm version: 3.2.8

Homeserver

Synapse 1.58.0

Will you send logs?

No

@robintown robintown added T-Defect S-Minor Impairs non-critical functionality or suitable workarounds exist A-Timeline A11y O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels May 9, 2022
@SimonBrandner
Copy link
Contributor

@nadonomy, do you think they should stand out more?

@nadonomy
Copy link
Contributor

Yeah, perhaps @gaelledel you could suggest a better colour variable for us to use from Compound which will look good on both light & dark?

@rufuskahler
Copy link

Removing 'Needs-Design' as will be resolved through EX work.

@t3chguy
Copy link
Member

t3chguy commented Jan 30, 2023

@rufuskahler removing X-Needs-Design means anyone can just start it with assets already provided in the issue, (e.g. community) - that does not appear to be the case - as without knowing how design expects this to look this will be impossible to contribute.

@germain-gg
Copy link
Contributor

Removing the a11y label, as it is not a requirement for those decorative lines to pass any contrast check.
This however could be a stylistic choice

@germain-gg germain-gg removed the A11y label Jul 19, 2023
@robintown robintown self-assigned this Feb 15, 2024
robintown added a commit to robintown/matrix-react-sdk that referenced this issue Mar 14, 2024
This is a reintroduction of matrix-org#12255, and should be the last part of the Compound color refinement work (element-hq/element-web#26992). I've updated the following components to reflect design decisions (https://www.figma.com/file/pqhlirZFEwEp5XpudSbiia/Theming-Web-templates?type=design&node-id=80-60219&mode=design&t=mjqwylUie1vRwzOV-0) that will help them fit in better with the new design system:

- Message bubbles
- Pills
- Sent indicators
- Jump to bottom/unread buttons
- Composer placeholder
- Notification badges
- Room sublists
- Verification shields
- Date separators
- Tooltips

Closes element-hq/element-web#22122
github-merge-queue bot pushed a commit to matrix-org/matrix-react-sdk that referenced this issue Mar 18, 2024
* Refine the colors of some more components

This is a reintroduction of #12255, and should be the last part of the Compound color refinement work (element-hq/element-web#26992). I've updated the following components to reflect design decisions (https://www.figma.com/file/pqhlirZFEwEp5XpudSbiia/Theming-Web-templates?type=design&node-id=80-60219&mode=design&t=mjqwylUie1vRwzOV-0) that will help them fit in better with the new design system:

- Message bubbles
- Pills
- Sent indicators
- Jump to bottom/unread buttons
- Composer placeholder
- Notification badges
- Room sublists
- Verification shields
- Date separators
- Tooltips

Closes element-hq/element-web#22122

* Update some screenshots
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Timeline O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Design Z-Confidence-High High confidence in the enhancement or suggestion based on known factors, or as described.
Projects
None yet
7 participants