Skip to content

Let the editor onboarding content follow the container background - #4485

Merged
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:fix-onboarding-background
Oct 13, 2026
Merged

vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:fix-onboarding-background

Conversation

@vogella

@vogella vogella commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

The onboarding controls of an empty editor stack copied the folder background into their composites once, at creation. Now that the CSS engine also styles non-page children of a CTabFolder, the onboarding container takes the theme color while the inner composite keeps the stale one, which shows up as a white box on the grey editor area with the Light theme on GTK. The container now forces its background onto its children, so the image, the text and the shortcut labels always match the editor area, including after a theme switch or a change of the active stack. A new StackRendererTest case covers it.

Fixes #4484

The onboarding controls of an empty editor stack copied the folder
background into their composites once, at creation. This went unnoticed
while the CSS engine skipped the container, but now that it styles every
non-page child of a CTabFolder, the container takes the theme color and
the inner composite keeps the stale one. With the Light theme on GTK
this showed a white box on the grey editor area.

The container now forces its background onto its children.

Fixes eclipse-platform#4484

Assisted-by: multiple AI agents and layers of automated tooling 🤖
@github-actions

Copy link
Copy Markdown
Contributor

Test Results

   867 files  ±0     867 suites  ±0   51m 17s ⏱️ - 10m 50s
 8 440 tests +1   8 198 ✅ +1  242 💤 ±0  0 ❌ ±0 
21 198 runs  +3  20 515 ✅ +3  683 💤 ±0  0 ❌ ±0 

Results for commit b73d26c. ± Comparison against base commit c8bd411.

@marcushoepfner marcushoepfner left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for fixing.

@vogella
vogella merged commit 964f079 into eclipse-platform:master Oct 13, 2026
18 checks passed
@vogella
vogella deleted the fix-onboarding-background branch October 13, 2026 08:28
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.

Default background color changed for "welcome" area content

2 participants