Skip to content

Conversation

@Pytal
Copy link
Member

@Pytal Pytal commented Nov 22, 2022

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
Copy link
Member Author

Pytal commented Nov 22, 2022

/backport to stable25

@nickvergessen
Copy link
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
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
Copy link
Member Author

Pytal commented Nov 25, 2022

@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

6 participants