Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 364879 - [Outline] Add Collapse All button
Summary: [Outline] Add Collapse All button
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-editor (show other bugs)
Version: 8.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 8.1.0   Edit
Assignee: Marc-André Laperle CLA
QA Contact: Anton Leherbauer CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-26 01:35 EST by Marc-André Laperle CLA
Modified: 2012-02-23 11:33 EST (History)
1 user (show)

See Also:
aleherb+eclipse: review+

Attachments
Add Collapse All button patch (3.90 KB, patch)
2011-12-21 19:15 EST, Marc-André Laperle CLA 		malaperle: iplog-
Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Marc-André Laperle CLA 2011-11-26 01:35:00 EST 
In the Outline view, it would be nice to have a "Collapse All" button like in JDT.
Comment 1 Marc-André Laperle CLA 2011-12-21 19:15:31 EST 
Created attachment 208714 [details]
Add Collapse All button patch

This patch adds the Collapse All button to the outline. Also, by handling org.eclipse.ui.handlers.CollapseAllHandler, the keybinding Ctrl+Shift+Numpad_Divide works.
Comment 2 Marc-André Laperle CLA 2011-12-21 19:17:20 EST 
Anton, would you mind reviewing the small patch? Thanks!
Comment 3 Anton Leherbauer CLA 2011-12-22 03:45:30 EST 
(In reply to comment #2)
> Anton, would you mind reviewing the small patch? Thanks!

Looks good. 
For an extra bonus, could you combine the two CollapseAllActions into one and put it into the org.eclipse.cdt.internal.ui.actions package (ideally moving messages as well)?
Thanks!
Comment 4 Marc-André Laperle CLA 2012-01-03 00:43:39 EST 
(In reply to comment #3)
> (In reply to comment #2)
> > Anton, would you mind reviewing the small patch? Thanks!
> 
> Looks good. 
> For an extra bonus, could you combine the two CollapseAllActions into one and
> put it into the org.eclipse.cdt.internal.ui.actions package (ideally moving
> messages as well)?
> Thanks!

I combined the two actions and moved the messages. I also updated the user documentation a little bit. Thank you for the review and suggestions!
Comment 5 Anton Leherbauer CLA 2012-01-03 06:07:04 EST 
Very nice!