Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 370742

Summary: No icons on the Debug view Layout action
Product: [Eclipse Project] Platform Reporter: Michael Rennie <Michael_Rennie>
Component: DebugAssignee: Pawel Piech <pawel.1.piech>
Status: VERIFIED FIXED QA Contact:
Severity: minor    
Priority: P3 CC: pawel.1.piech
Version: 3.8Keywords: polish
Target Milestone: 3.8 M6   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Attachments:
Description Flags
bp view with action icons
none
debug view with no icons
none
Tree mode icon
none
Breadcrumb mode icon.
none
Auto mode icon.
none
screen shot none

Description Michael Rennie CLA 2012-02-06 11:37:10 EST
Created attachment 210601 [details]
bp view with action icons

Version: 4.2.0
Build id: I20120126-1300

Unlike the rest of the debug views, the Debug View > Layout actions do not have the associated icons on the actions.

See the attached screen shots
Comment 1 Michael Rennie CLA 2012-02-06 11:37:58 EST
Created attachment 210602 [details]
debug view with no icons
Comment 2 Michael Rennie CLA 2012-02-06 11:43:10 EST
The problem is that we do not set the image descriptor in DebugViewModeAction
Comment 3 Pawel Piech CLA 2012-02-22 12:07:16 EST
Created attachment 211426 [details]
Tree mode icon
Comment 4 Pawel Piech CLA 2012-02-22 12:07:37 EST
Created attachment 211427 [details]
Breadcrumb mode icon.
Comment 5 Pawel Piech CLA 2012-02-22 12:08:28 EST
Created attachment 211428 [details]
Auto mode icon.

Given my limited artistic abilities.  Here are the icons I hacked up for the debug view layout menu.  Mike is this what you had in mind?
Comment 6 Michael Rennie CLA 2012-02-22 14:06:36 EST
Created attachment 211440 [details]
screen shot

looks good to me.

Since I have the changes in my workspace do you want me to push it for you?
Comment 7 Pawel Piech CLA 2012-02-22 14:24:17 EST
Sure, I had only drawn the icons so far, but if you hooked them up already then go ahead :-)
Comment 8 Pawel Piech CLA 2012-02-22 14:25:28 EST
(In reply to comment #6)
> Created attachment 211440 [details]
> screen shot
Oh I see, you just re-used the detail pane icons instead of the ones I attached.  That's fine by me too.
Comment 10 Michael Rennie CLA 2012-03-14 12:12:56 EDT
Verified in:
Version: 4.2.0
Build id: I20120313-0610