Skip to content

Give FormHeading a distinct background in the light theme#3950

Merged
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:light-theme-form-chrome
May 8, 2026
Merged

Give FormHeading a distinct background in the light theme#3950
vogella merged 1 commit into
eclipse-platform:masterfrom
vogella:light-theme-form-chrome

Conversation

@vogella
Copy link
Copy Markdown
Contributor

@vogella vogella commented Apr 30, 2026

The light theme set both Form and FormHeading to #ffffff plus a matching text-background-color, so the form heading area (the bar containing the editor title and toolbar icons) was visually indistinguishable from the form body and the icons looked free-floating.

Split the joint Form, FormHeading rule in e4-light_globalstyle.css and give FormHeading (and the Form's text-background-color) #eaeaea — the same color the Section title bars already use — so the heading reads as a distinct band against the white body.

@BeckerWdf
Copy link
Copy Markdown
Member

Just one question that just now comes to my mind:
Was the removal of the grey background a side-effekt of your change (where the gradients were removed). Or was this a change @mvm-sap made by intention during her work on the light theme?

@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented Apr 30, 2026

Or was this a change @mvm-sap made by intention during her work on the light theme?

All I can say that Version: 2026-03 (4.39) does NOT have the gray background. Why? I don't know and don't have the interest to research that.

@BeckerWdf
Copy link
Copy Markdown
Member

Why? I don't know and don't have the interest to research that.

Let me have a look if I can find out something...

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 30, 2026

Test Results

   858 files  +   20     858 suites  +20   51m 25s ⏱️ + 14m 9s
 7 977 tests ±    0   7 733 ✅ ±    0  243 💤 ± 0  1 ❌ ±0 
20 433 runs  +1 070  19 777 ✅ +1 012  655 💤 +58  1 ❌ ±0 

For more details on these failures, see this check.

Results for commit a93ab2a. ± Comparison against base commit 774c047.

♻️ This comment has been updated with latest results.

@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented May 7, 2026

@BeckerWdf any news? I think M3 closes soon, so you to decide if you want this in the next release or not, I do not have a strong opinion here.

@BeckerWdf
Copy link
Copy Markdown
Member

Why? I don't know and don't have the interest to research that.

Let me have a look if I can find out something...

We did not remove the header styling by intention...

@BeckerWdf
Copy link
Copy Markdown
Member

So I have no objections to give the header that grey background

@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented May 7, 2026

No objection is not sufficient to merge. I also have no objections .;-)

Just decide: yes / no

@BeckerWdf
Copy link
Copy Markdown
Member

yes

@vogella vogella marked this pull request as ready for review May 8, 2026 07:03
The as-shipped light theme set both Form and FormHeading to #ffffff
plus a matching text-background-color, so the form heading area was
visually indistinguishable from the form body.

Split the joint Form/FormHeading rule, set FormHeading and the Form's
text-background-color to #eaeaea (matching the Section title bar) so
the heading stands out against the white body.
@vogella vogella force-pushed the light-theme-form-chrome branch from be3acaf to a93ab2a Compare May 8, 2026 07:08
@vogella
Copy link
Copy Markdown
Contributor Author

vogella commented May 8, 2026

Unrelated unit test error.

@vogella vogella merged commit 25b0279 into eclipse-platform:master May 8, 2026
16 of 18 checks passed
@vogella vogella deleted the light-theme-form-chrome branch May 8, 2026 08:54
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