Skip to content

[target] Remove Content and Environment Section from the Definition page#1826

Merged
laeubi merged 1 commit intoeclipse-pde:masterfrom
laeubi:remove_sections_from_Definition_page
Jun 19, 2025
Merged

[target] Remove Content and Environment Section from the Definition page#1826
laeubi merged 1 commit intoeclipse-pde:masterfrom
laeubi:remove_sections_from_Definition_page

Conversation

@laeubi
Copy link
Copy Markdown
Contributor

@laeubi laeubi commented Jun 19, 2025

Currently the Target-Editor contains two sections "Content" and "Environment" that roughly explain what these things can be used for if one clicks on the corresponding tabs. In practice, these information is barley usable for todays work an users uncertain would better consult the Eclipse Help or read a tutorial on PDE. More worse, these clutter the UI and steal valuable space not available for the actual content of the location leading to scrollbars appearing.

This now removes the sections entirely ro reclaim the space for something the user is actually interested in.

Before

grafik

After

grafik

Currently the Target-Editor contains two sections "Content" and
"Environment" that roughly explain what these things can be used for if
one clicks on the corresponding tabs. In practice, these information is
barley usable for todays work an users uncertain would better consult
the Eclipse Help or read a tutorial on PDE. More worse, these clutter
the UI and steal valuable space not available for the actual content of
the location leading to scrollbars appearing.

This now removes the sections entirely ro reclaim the space for
something the user is actually interested in.
@akurtakov
Copy link
Copy Markdown
Member

Really good call! More content is better than such labels.

@github-actions
Copy link
Copy Markdown

Test Results

   765 files  +  206     765 suites  +206   56m 12s ⏱️ + 12m 8s
 3 611 tests ±    0   3 535 ✅ +    2   76 💤  -   2  0 ❌ ±0 
10 833 runs  +3 248  10 602 ✅ +3 143  231 💤 +105  0 ❌ ±0 

Results for commit c756709. ± Comparison against base commit 1de98b9.

@laeubi laeubi merged commit 9ed56a6 into eclipse-pde:master Jun 19, 2025
19 checks passed
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