- 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
over 1 year 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
11 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
6 months 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
6 months ago 9 pass - last update
6 months ago 9 pass Updated the patch from @ni.pineau@gmail.com with updated border color when hover and active.
- last update
6 months ago 9 pass - last update
6 months 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
6 months ago 1:42pm 20 June 2024 - Issue was unassigned.
- Status changed to RTBC
5 months ago 6:13am 24 July 2024 - 🇮🇳India Tirupati_Singh
I've applied the MR!33 as patch and it applied successfully with no errors. After applying patch, the issue of jumping color boxes on hover has been resolved and it is working fine. The active/selected color box can be identified clearly. The MR resolved the issue hence, moving the issue status to Reviewed & tested. Attaching the before and after fixes attachments for reference.
- 🇵🇱Poland bronismateusz
@tirupati_singh Nice work, your merge request works as expected with Drupal 11.0.7.