[5532] UX: Improve accessibility of default theme - colour palette#5613
Merged
[5532] UX: Improve accessibility of default theme - colour palette#5613
Conversation
knolleary
reviewed
Apr 13, 2026
Member
knolleary
left a comment
There was a problem hiding this comment.
In general, looks good. Two items that don't quite work for me yet.
- The contrast of selected-vs-unselected toggle buttons - for eg the tab buttons:
- The selected node appearance. The blue takes getting used to, which I'm prepared to do - but I think it needs something else. I'm going to experiment on this and share some ideas.
Member
Member
|
I've updated the selected node appearance via #5623 (without the colour changes). Once that's in, lets update this branch and then we can do battle over which shade of blue we use. |
Contributor
Author
|
Hey @knolleary these are both addressed!
For 2 - I went ahead and changed the |
Member
Member
|
I spotted the fix from #5607 was applying to all tabs and needed scoping to just the workspace tabs. Have pushed that fix here. |
Fix node halo height calculation when status is shown
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.


Types of changes
Proposed changes
Resolves #5532
See details of changes here.
Checklist
npm run testto verify the unit tests pass