Skip to content

[UI] Fix scope bar text unreadable in dark mode - #26

Merged
hannesa2 merged 1 commit into
gitx:masterfrom
goneng:dark-mode-scope-bar
Oct 3, 2026
Merged

hannesa2 merged 1 commit into
gitx:masterfrom
goneng:dark-mode-scope-bar

Conversation

@goneng

@goneng goneng commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Draw the bar in dark colours under a dark appearance. Until now it
always painted its light-grey gradient, while its buttons' titles turn
light in dark mode, so they were barely readable. In GitX this is the
Source / Blame / History bar above the file view.

Changes

  • Give the background gradient, border, separator and label colours a
    dark-appearance value, resolved when the bar draws
  • Keep every light-appearance value as it was
  • Test the background brightness under Aqua and Dark Aqua

Screenshots

Before (Dark Mode)

image

After (Dark Mode)

image

After (Light Mode)

image

Test plan

  • swift test: the Dark Aqua test fails before the change and passes
    after it
  • Offscreen render of a filled bar (radio group, labelled group,
    separator) built from the old and new sources: light mode is
    byte-identical, dark mode now shows a dark bar with readable titles

@goneng
goneng marked this pull request as ready for review September 30, 2026 21:21
Draw the bar in dark colours under a dark appearance, so its buttons,
whose titles already turn light there, stay readable.

- Give the background gradient, border, separator and label colours a
  dark-appearance value, resolved when the bar draws
- Keep every light-appearance value as it was
- Test the background brightness under Aqua and Dark Aqua
@goneng
goneng force-pushed the dark-mode-scope-bar branch from df5280d to b3bfed7 Compare September 30, 2026 21:25
@goneng goneng changed the title Fix scope bar text unreadable in dark mode [UI] Fix scope bar text unreadable in dark mode Oct 2, 2026
@goneng

goneng commented Oct 2, 2026

Copy link
Copy Markdown
Collaborator Author

@hannesa2, this fixes another UI bug that is related to dark-mode support - pls review

@goneng
goneng requested a review from hannesa2 October 3, 2026 07:19
@hannesa2
hannesa2 merged commit 2671a7e into gitx:master Oct 3, 2026
1 check passed
@hannesa2

hannesa2 commented Oct 3, 2026

Copy link
Copy Markdown

Btw, I invited you here too

@goneng

goneng commented Oct 3, 2026

Copy link
Copy Markdown
Collaborator Author

Btw, I invited you here too

Thanks! - let's see if this fix works... (I will rebase/update the corresponding PR in GitX)

@goneng
goneng deleted the dark-mode-scope-bar branch October 3, 2026 07:55
hannesa2 pushed a commit to gitx/gitx that referenced this pull request Oct 3, 2026
Pick up the MGScopeBar fix that draws the bar above the file view in
dark colours under a dark appearance (gitx/MGScopeBar#26).

- Bump External/MGScopeBar to 2671a7e, which contains only that fix
- Light mode is unchanged: the bar renders byte-identical to before
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants