Skip to content

Fix border color contrast#35315

Merged
Pytal merged 1 commit into
masterfrom
enh/a11y-border-contrast
Nov 22, 2022
Merged

Fix border color contrast#35315
Pytal merged 1 commit into
masterfrom
enh/a11y-border-contrast

Conversation

@Pytal

@Pytal Pytal commented Nov 22, 2022

Copy link
Copy Markdown
Member

Border color contrast requirements

To meet WCAG AA contrast requirements for border colors, categorized under “Graphical Objects and User Interface Components”, a minimum 3:1 foreground-background contrast ratio is required

Primary color borders

Original color is lightened slightly
#00639a#006aa3

Before/After contrast ratios

Dark mode

Before After
image image

Light mode

Before After
image image

General-purpose borders

Dark mode
--color-border: #292929#646464
--color-border-dark: #3b3b3b#787878 (greater contrast on dark backgrounds due to being lighter than --color-border)

Light mode
--color-border: #ededed#949494
--color-border-dark: #dbdbdb#808080 (greater contrast on light backgrounds due to being darker than --color-border)

Before/After contrast ratios

Dark mode

Before After
image image

Light mode

Before After
image image

Examples

Primary + general border colors

Before After
image image

General border color usage

Before After
image image

@Pytal Pytal added 3. to review Waiting for reviews accessibility labels Nov 22, 2022
@Pytal Pytal added this to the Nextcloud 26 milestone Nov 22, 2022
@Pytal Pytal self-assigned this Nov 22, 2022
@Pytal

Pytal commented Nov 22, 2022

Copy link
Copy Markdown
Member Author

/backport to stable25

@nickvergessen

Copy link
Copy Markdown
Member

OCA\Theming\Tests\Service\DefaultThemeTest::testThemindDisabledFallbackCss
Failed asserting that two strings are equal.

Tests fail

Signed-off-by: Christopher Ng <chrng8@gmail.com>
@Pytal Pytal force-pushed the enh/a11y-border-contrast branch from 4d2a9b1 to a0266d1 Compare November 22, 2022 18:43
@Pytal Pytal added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Nov 22, 2022
@Pytal Pytal merged commit 53195cb into master Nov 22, 2022
@Pytal Pytal deleted the enh/a11y-border-contrast branch November 22, 2022 23:19
@PVince81

Copy link
Copy Markdown
Member

@Pytal theming tests are failing on master now, please fix them. I'll ignore them for now as we need to ship the RC soon.

@Pytal

Pytal commented Nov 25, 2022

Copy link
Copy Markdown
Member Author

@Pytal theming tests are failing on master now, please fix them. I'll ignore them for now as we need to ship the RC soon.

Rather, master is passing e.g. https://drone.nextcloud.com/nextcloud/server/26432 and CI for this PR was all green in https://drone.nextcloud.com/nextcloud/server/26173

They are failing on stable25 since #35346 in https://drone.nextcloud.com/nextcloud/server/26177

Fixed in #35415

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish accessibility

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BITV] Dark mode borders need higher contrast

5 participants