- Issue created by @rkoller
- Merge request !12420changed the border color to increase the contrast ratio. โ (Closed) created by libbna
- ๐ฎ๐ณIndia libbna New Delhi, India
I've updated the border color from #dedfe4 to #949494. The border is now much more visually distinct.
- ๐ฎ๐ณIndia roshanibhangale
Hi
I have tested the issue on the Drupal 11.x
After MR3986 is applied successfully, Add fields has the border is now dark and border color is updated to #949494.
Hence this can be move to RTBC +1
Attaching screenshot for reference.
- ๐ฌ๐งUnited Kingdom the_g_bomb
Confirmed, patch applies and improves the border contrast.
I have also checked the new contrast with a contrast Checker.
- ๐ฉ๐ชGermany rkoller Nรผrnberg, Germany
thanks for the MR! i agree #949494 has a high enough contrast, but problem is that this color is not one of the available color variations for claro. the list of available variables is:
/* Gray variations. */ --color-gray: #232429; --color-gray-900: #393a3f; --color-gray-800: #55565b; --color-gray-700: #75767b; --color-gray-600: #828388; --color-gray-500: #919297; --color-gray-400: #adaeb3; --color-gray-300: #c1c2c7; --color-gray-200: #d3d4d9; --color-gray-100: #dedfe4; --color-gray-050: #f3f4f9; --color-gray-025: #f9faff;
- ๐ฎ๐ณIndia libbna New Delhi, India
I checked using the contrast checker, and
--color-gray-500: #919297;
meets the expected contrast ratio. - ๐ฎ๐ณIndia libbna New Delhi, India
Forgot to attached the screenshot of the final output.
- ๐ฉ๐ชGermany rkoller Nรผrnberg, Germany
thank you! I've updated the issue summary to reflect the ui changes. and as said in the issue summary the contrast of 3.1:1 is meeting the requirement. so looks all good now.
- ๐ฏ๐ตJapan neptune-dc
I have confirmed that the new border color is being applied at `admin/structure/types/manage/article/fields` > Create a New Field. +1 RBTC
-
longwave โ
committed dadcf295 on 11.2.x
Issue #3530855 by libbna, rkoller, the_g_bomb: The border of field types...
-
longwave โ
committed dadcf295 on 11.2.x
-
longwave โ
committed e7013351 on 11.x
Issue #3530855 by libbna, rkoller, the_g_bomb: The border of field types...
-
longwave โ
committed e7013351 on 11.x
- ๐ฌ๐งUnited Kingdom longwave UK
This is eligible for backport as an accessibility bug fix, but does not apply cleanly to 10.6.x. If anyone wants to backport this down to 10.x please open a backport MR; otherwise we will just fix this in 11.x and move on.
Committed and pushed e7013351307 to 11.x and dadcf295d0c to 11.2.x. Thanks!
For changes like this, screenshots are only helpful if they show both before and after, so I have not credited anyone that only uploaded "after" screenshots.
Automatically closed - issue fixed for 2 weeks with no activity.