Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Active Part Indicator to Light Theme #1652

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

BeckerWdf
Copy link
Contributor

The dark theme already has such an indicator. It adds a blue underline on the active tab.

This change aligns the dark/light theme and adds the blue underline also to the light theme.

Fixes: #1651

@BeckerWdf
Copy link
Contributor Author

This is how it looks like:
image

The blue is not the same as in the dark theme. Actually I took the dark blue of the gradient that is used (on macOS) in the classic theme.

@BeckerWdf
Copy link
Contributor Author

@HeikoKlare: Would be great if you could find some time to test this on windows.

Copy link
Contributor

github-actions bot commented Feb 7, 2024

Test Results

   917 files  +   611     917 suites  +611   38m 5s ⏱️ + 22m 56s
 7 433 tests ±     0   7 282 ✅ +     7  150 💤  -   8  1 ❌ +1 
21 876 runs  +14 060  21 492 ✅ +13 834  383 💤 +225  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 13a18b7. ± Comparison against base commit d7676d3.

♻️ This comment has been updated with latest results.

The dark theme already has such an indicator. It adds a blue underline
on the active tab.

This change aligns the dark/light theme and adds the blue underline
also to the light theme.

Fixes: eclipse-platform#1651
@BeckerWdf
Copy link
Contributor Author

Failing test is not related to this change. Already reported in #1005

@shubhamWaghmare-sap
Copy link
Contributor

Performed some basic manual tests for this change on Windows. The active tab is highlighted with a blue underline in both the view and editor areas, based on selection.
The fix looks good to me.

2024-02-08_15-56-34

@BeckerWdf
Copy link
Contributor Author

Performed some basic manual tests for this change on Windows. The active tab is highlighted with a blue underline in both the view and editor areas, based on selection. The fix looks good to me.

2024-02-08_15-56-34 2024-02-08_15-56-34

Thanks for testing

@BeckerWdf BeckerWdf merged commit 6d0c74a into eclipse-platform:master Feb 8, 2024
14 of 16 checks passed
@BeckerWdf BeckerWdf deleted the active_tab_indicator branch February 8, 2024 12:08
@HeikoKlare
Copy link
Contributor

Sorry for the late reply. Since test on Windows was already done, I can at least contribute by confirming that it also works on GTK:

image

Great improvement. Thanks, Matthias!

BeckerWdf added a commit to BeckerWdf/www.eclipse.org-eclipse that referenced this pull request Feb 13, 2024
BeckerWdf added a commit to eclipse-platform/www.eclipse.org-eclipse that referenced this pull request Feb 14, 2024
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.

Active view/part indicator consistency in light theme
3 participants