- Issue created by @dawid_nawrot
- ๐ฎ๐ณIndia snehal-chibde
hello @david_nawrot, applied the above patch and it is working as expected.
- Status changed to Needs review
10 months ago 12:20pm 22 August 2023 Thank you very much that's really great. One tiny comment though : it's not easy to spot on which color is currently selected because the shadow color is quite light. Do you think you could choose a color that would be a bit darker ?
Thanks again.
- last update
5 months ago Patch Failed to Apply Patch from @snehal-chibde with just active shadow color updated.
- ๐ซ๐ทFrance vanessa.fayard
Hello,
Patches 1 and 2 successfully tested and validated.
Proposal 2 approved to enhance contrast and readability of active element.Patch 1 by @david_nawrot
Patch 2 by @n.pineau
Arijit Acharya โ made their first commit to this issueโs fork.
- Assigned to Arijit Acharya
- Status changed to Needs work
3 days ago 12:22pm 20 June 2024 The patch applied is fine. But as a user I think the border color is still not eye catching, probably a prominent black color in place of grey would be more soothing to the eyes. I leave it on the maintainer of this module to decide it.
Moving this to Needs work state. Also attaching a MR in some time for the fix.
Thank You.- Merge request !333372480-jumping-color-boxes: Changed the boder color to black on hover and focus. โ (Open) created by Arijit Acharya
- last update
3 days ago 9 pass - last update
3 days ago 9 pass Updated the patch from @ni.pineau@gmail.com with updated border color when hover and active.
- last update
3 days ago 9 pass - last update
3 days ago 9 pass Sorry for the multiple commits. As stylelint was not setup in the local so took the extra commits.
- Status changed to Needs review
3 days ago 1:42pm 20 June 2024 - Issue was unassigned.