Skip to content

Add expand and collapse options to API Errors/Warnings preferences page #2117

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.

Sign up for GitHub

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

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

Add expand and collapse options to API Errors/Warnings preferences page #2117

elsazac wants to merge 1 commit into from

Conversation

@elsazac
Copy link
Contributor

@elsazac elsazac commented Nov 11, 2025 •
edited
Loading

This change introduces Expand All and Collapse All toolbar options to the API Errors/Warnings preference page under Settings → Plugin Development Environment → API Errors/Warnings. These options make it easier to navigate the large number of expandable sections on the page, aligning the behaviour with other preference pages such as Java → Compiler → Errors/Warnings.

newCE1.mov

@github-actions
Copy link

github-actions bot commented Nov 11, 2025 •
edited
Loading

Test Results

  623 files   -   148    623 suites   - 148   54m 32s ⏱️ - 1m 26s
3 648 tests ±    0  3 593 ✅  -     1   54 💤 ± 0  1 ❌ +1 
8 213 runs   - 2 665  8 073 ✅  - 2 642  139 💤  - 24  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 4285494. ± Comparison against base commit abc3cc7.

♻️ This comment has been updated with latest results.

@elsazac elsazac force-pushed the expand_collapse_Bar_in_UI branch from 8c4d0bc to e30781c Compare November 11, 2025 11:58
@deepika-u
Copy link
Contributor

deepika-u commented Nov 11, 2025

@elsazac : I like the thought. Good one.

gireeshpunathil
gireeshpunathil reviewed Nov 12, 2025
View reviewed changes
...e.api.tools.ui/src/org/eclipse/pde/api/tools/ui/internal/preferences/PreferenceMessages.java Outdated
Comment on lines 168 to 169
public static String Preference_Collapse_All;
public static String Preference_Expand_All;
Copy link
Contributor

@gireeshpunathil gireeshpunathil Nov 12, 2025

Choose a reason for hiding this comment

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

Suggested change
public static String Preference_Collapse_All;
public static String Preference_Expand_All;
public static String PREFERENCE_COLLAPSE_ALL;
public static String PREFERENCE_EXPAND_ALL;

Copy link
Contributor Author

@elsazac elsazac Nov 12, 2025

Choose a reason for hiding this comment

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

Done.

...tools.ui/src/org/eclipse/pde/api/tools/ui/internal/preferences/preferencemessages.properties Outdated
Comment on lines 142 to 143
Preference_Collapse_All= Collapse All
Preference_Expand_All= Expand All
Copy link
Contributor

@gireeshpunathil gireeshpunathil Nov 12, 2025

Choose a reason for hiding this comment

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

Suggested change
Preference_Collapse_All= Collapse All
Preference_Expand_All= Expand All
Preference_Collapse_All= Collapse all preferences
Preference_Expand_All= Expand all preferences

Copy link
Contributor Author

@elsazac elsazac Nov 12, 2025

Choose a reason for hiding this comment

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

Done.

@elsazac elsazac force-pushed the expand_collapse_Bar_in_UI branch 2 times, most recently from 8d7a015 to 6b291ac Compare November 12, 2025 05:23
@elsazac
Copy link
Contributor Author

elsazac commented Nov 13, 2025

@laeubi Could you check on this?

@elsazac elsazac force-pushed the expand_collapse_Bar_in_UI branch from 6b291ac to bb05c4a Compare November 13, 2025 04:02
@elsazac elsazac mentioned this pull request Nov 13, 2025
Open
@elsazac elsazac force-pushed the expand_collapse_Bar_in_UI branch from bb05c4a to 3535a4d Compare December 15, 2025 13:42
@eclipse-pde-bot
Copy link
Contributor

eclipse-pde-bot commented Dec 15, 2025 •
edited
Loading

This pull request changes some projects for the first time in this development cycle.
Therefore the following files need a version increment:

apitools/org.eclipse.pde.api.tools.ui/META-INF/MANIFEST.MF

An additional commit containing all the necessary changes was pushed to the top of this PR's branch. To obtain these changes (for example if you want to push more changes) either fetch from your fork or apply the git patch.

Git patch 
From 8337342d6df5e9db5bf953a87d63937cbc8f48f1 Mon Sep 17 00:00:00 2001
From: Eclipse PDE Bot <[email protected]>
Date: Tue, 16 Dec 2025 14:16:54 +0000
Subject: [PATCH] Version bump(s) for 4.39 stream


diff --git a/apitools/org.eclipse.pde.api.tools.ui/META-INF/MANIFEST.MF b/apitools/org.eclipse.pde.api.tools.ui/META-INF/MANIFEST.MF
index 184c5d0bd9..e1ea8773c5 100644
--- a/apitools/org.eclipse.pde.api.tools.ui/META-INF/MANIFEST.MF
+++ b/apitools/org.eclipse.pde.api.tools.ui/META-INF/MANIFEST.MF
@@ -2,7 +2,7 @@ Manifest-Version: 1.0
 Bundle-ManifestVersion: 2
 Bundle-Name: %pluginName
 Bundle-SymbolicName: org.eclipse.pde.api.tools.ui; singleton:=true
-Bundle-Version: 1.4.200.qualifier
+Bundle-Version: 1.4.300.qualifier
 Bundle-Localization: plugin
 Bundle-ActivationPolicy: lazy
 Bundle-Vendor: %providerName
-- 
2.52.0

Further information are available in Common Build Issues - Missing version increments.

@elsazac elsazac force-pushed the expand_collapse_Bar_in_UI branch 3 times, most recently from 5902ee3 to 210b7a4 Compare December 16, 2025 14:13
@elsazac
This change introduces "Expand All" and "Collapse All" toolbar option
4285494 
in the API Errors/Warnings preference page in the UI
(Eclipse->Settings->Plugin-Development Environment->API
Errors/Warnings)
corresponding version bump done
@elsazac elsazac force-pushed the expand_collapse_Bar_in_UI branch from d033a7a to 4285494 Compare December 16, 2025 14:23
@elsazac
Copy link
Contributor Author

elsazac commented Dec 16, 2025

@vogella would it be possible for you to merge this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gireeshpunathil gireeshpunathil gireeshpunathil approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@elsazac @deepika-u @eclipse-pde-bot @gireeshpunathil