Skip to content

Style the progress bar in the dark theme - #4319

Open
vogella wants to merge 1 commit into
eclipse-platform:masterfrom
vogella:dark-theme-progress-bar
Open

Style the progress bar in the dark theme#4319
vogella wants to merge 1 commit into
eclipse-platform:masterfrom
vogella:dark-theme-progress-bar

Conversation

@vogella

@vogella vogella commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

The progress bar had no rule of its own in the dark theme: only the ProgressIndicator around it was coloured, so the bar fell to the Composite > * catch-all on DARK_BACKGROUND and did not separate from the chrome.

Gives it #5A5F62 as the trough and LINK_COLOR as the fill. The .MPart copies live in e4-dark_partstyle.css because they only tie .MPart Composite > * > * on specificity and need to be imported after it.

Verified on Windows in the Progress view, the status line and the p2 install wizard: fill and trough resolve to the two colours at every fill level.

🤖 Generated with Claude Code

@vogella

vogella commented Aug 31, 2026

Copy link
Copy Markdown
Contributor Author

Windows neu:

styled-progress-bars10 styled-progress-bars20

@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Test Results

   858 files  ± 0     858 suites  ±0   1h 1m 23s ⏱️ + 23m 12s
 8 302 tests ± 0   8 059 ✅ ± 0  243 💤 ± 0  0 ❌ ±0 
20 775 runs  +86  20 105 ✅ +63  670 💤 +23  0 ❌ ±0 

Results for commit 2e56600. ± Comparison against base commit 94cf8a1.

♻️ This comment has been updated with latest results.

@BeckerWdf

Copy link
Copy Markdown
Member

how did it look before?

@vogella
vogella force-pushed the dark-theme-progress-bar branch from b6f580a to 80a0518 Compare September 1, 2026 15:41
The bar had no rule of its own: only the ProgressIndicator around it was
coloured, so the bar fell to the 'Composite > *' catch-all on
DARK_BACKGROUND and did not separate from the chrome.

Give it #5A5F62 as the trough and LINK_COLOR as the fill. The '.MPart'
copies live in e4-dark_partstyle.css because they only tie
'.MPart Composite > * > *' on specificity and need to be imported after it.

Assisted-by: multiple AI agents and layers of automated tooling 🤖
@vogella
vogella force-pushed the dark-theme-progress-bar branch from 80a0518 to 2e56600 Compare September 8, 2026 06:56
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