👓 a11y: Add Solid Marker to Improve Visibility in LLM Menu #7714
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Originally #7686
In testing we found that users who have low visions and use assistive technology had trouble identifying the correct LLM models in the submenu due to the low contrast between the active-item and surrounding background color. This PR adds a solid black indicator to the left border edge to indicate the current active item in sub-menus to add another visual cue to support people with low vision and making it easier to distinguish the current active item in the menu from other items.
Screen Recording of the submenu chages in dark and light mode.
This pull request addresses issue #7608 with the style outlined in the issue and in discussion of PR #6645.
Files touched:
client/src/components/Chat/Menus/Endpoints/CustomMenu.tsx @ line 162.
Added Tailwind classes:
data-[active-item]:border-l-2
-- left border, 2px widthdata-[active-item]:rounded-l-none
-- no border-radius on leftdata-[active-item]:border-black
-- border color is blackdark:data-[active-item]:border-white
- change border color for dark modeChange Type
Please delete any irrelevant options.
Testing
Manual keyboard testing.
Test Configuration:
MacOS Sequoia 15.3.2
Chrome/Safari/Firefox
Checklist
Please delete any irrelevant options.