Bugzilla – Full Text Bug Listing
| Summary: | [Outline] Add Collapse All button | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Tools] CDT | Reporter: | Marc-André Laperle <malaperle> | ||||
| Component: | cdt-editor | Assignee: | Marc-André Laperle <malaperle> | ||||
| Status: | RESOLVED FIXED | QA Contact: | Anton Leherbauer <aleherb+eclipse> | ||||
| Severity: | enhancement | ||||||
| Priority: | P3 | CC: | cdtdoug | ||||
| Version: | 8.0 | Flags: | aleherb+eclipse:
review+
|
||||
| Target Milestone: | 8.1.0 | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
|
Description
Marc-André Laperle
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.
Anton, would you mind reviewing the small patch? Thanks! (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! (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! Very nice! |