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

Plug-In Image browser does not highlight the current page in dark theme #904

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lathapatil
Copy link
Contributor

Temporary fix to disable the styling for current page (Hyperlink widget) and setting white color to it (only if current theme is Dark Theme ).

Fixes #822

Copy link

github-actions bot commented Nov 10, 2023

Test Results

   279 files     279 suites   53m 49s ⏱️
 3 498 tests  3 440 ✅  58 💤 0 ❌
10 791 runs  10 615 ✅ 176 💤 0 ❌

Results for commit 9dbbe52.

♻️ This comment has been updated with latest results.

@lathapatil lathapatil force-pushed the Issues/822_PageNavigation_DarkThemeIssue branch from f1ab46e to 3337ee3 Compare November 15, 2023 11:20
@lathapatil
Copy link
Contributor Author

@vogella could you review the code and check if this fix is required here ?

@lathapatil lathapatil force-pushed the Issues/822_PageNavigation_DarkThemeIssue branch from 3337ee3 to 65e8b4f Compare December 7, 2023 08:41
Temporary fix to disable the styling for current page (Hyperlink widget)
and setting white color to it (only if current theme is Dark Theme ).

Fixes eclipse-pde#822
@laeubi laeubi force-pushed the Issues/822_PageNavigation_DarkThemeIssue branch from 65e8b4f to 9dbbe52 Compare January 27, 2024 15:59
@laeubi laeubi requested a review from vogella April 29, 2024 07:33
@laeubi
Copy link
Contributor

laeubi commented Apr 29, 2024

@vogella can you please review?
@lathapatil can you please provide screenshot before/after for light and for dark theme?

@lathapatil
Copy link
Contributor Author

lathapatil commented Apr 29, 2024

@lathapatil can you please provide screenshot before/after for light and for dark theme?

Before fix : Dark theme

2024-04-29_16h02_47

After Fix : Dark theme

image

Light Theme (No change after fix)

image

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.

Plug-In Image browser does not highlight the current page in dark theme
3 participants