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

Improve contrast in text selections #572

Merged
merged 3 commits into from
Jan 3, 2020
Merged

Conversation

danirabbit
Copy link
Member

Contrast here goes from 2.8:1 to 8.8:1 in the light style and from 2.8:1 to 5.1:1 in the dark style. So oddly the effect looks less jarring but is way more contrasty

BEFORE

Screenshot from 2020-01-02 10 15 20@2x

Screenshot from 2020-01-02 10 33 03@2x

AFTER

Screenshot from 2020-01-02 10 13 57@2x

Screenshot from 2020-01-02 10 25 23@2x

@danirabbit danirabbit changed the title Make selection more like a highlighter Improve contrast in text selections Jan 2, 2020
@cassidyjames
Copy link
Contributor

cassidyjames commented Jan 2, 2020

Should we effectively deprecate colorAccent in favor of accent_color_500? And define the former as the latter to ensure they stay in sync?

@danirabbit
Copy link
Member Author

@cassidyjames yeah I think that makes sense

@danirabbit
Copy link
Member Author

@cassidyjames I think we should address that in a different PR because it would require reworking a lot of stuff right now. That would significantly change colors in things and I think some of the gradients would be not so nice

@cassidyjames cassidyjames merged commit e812c60 into master Jan 3, 2020
@cassidyjames cassidyjames deleted the subtle-selection branch January 3, 2020 03:23
worldofpeace added a commit to worldofpeace/nixpkgs that referenced this pull request Jan 21, 2020
* Improve contrast in text selections elementary/stylesheet#572
* Add border and background to selected image radios elementary/stylesheet#576
dtzWill pushed a commit to dtzWill/nixpkgs that referenced this pull request Jan 28, 2020
* Improve contrast in text selections elementary/stylesheet#572
* Add border and background to selected image radios elementary/stylesheet#576

(cherry picked from commit dc18433)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants