Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 325767 - Legend dialog is incorrect
Summary: Legend dialog is incorrect
Status: RESOLVED FIXED
Alias: None
Product: PTP
Classification: Tools
Component: Debug UI (show other bugs)
Version: 4.0.3   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Beth Tibbitts CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-20 11:07 EDT by Beth Tibbitts CLA
Modified: 2012-12-11 17:02 EST (History)
1 user (show)

See Also:


Attachments
dialog showing the duplicate/missing icons (345.16 KB, image/png)
2010-09-20 11:07 EDT, Beth Tibbitts CLA
no flags Details
Old original legend showing (presumably) what's missing (368.56 KB, image/png)
2010-09-20 11:09 EDT, Beth Tibbitts CLA
no flags Details
Here's what it looks like now, as fixed (135.35 KB, image/png)
2010-09-20 13:43 EDT, Beth Tibbitts CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Beth Tibbitts CLA 2010-09-20 11:07:11 EDT
Created attachment 179252 [details]
dialog showing the duplicate/missing icons

Legend dialog has some missing icons and some uninteded repetition.

  Node Icons: includes two up's, two down's, two errors
(see attachment legend_error.png
Comment 1 Beth Tibbitts CLA 2010-09-20 11:09:34 EDT
Created attachment 179254 [details]
Old original legend showing (presumably) what's missing

Old original legend is attached.
Comment 2 Beth Tibbitts CLA 2010-09-20 11:40:30 EDT
fix the obviously incorrect ones in ptp_40 branch. will do HEAD too.
Comment 3 Beth Tibbitts CLA 2010-09-20 13:43:20 EDT
Created attachment 179261 [details]
Here's what it looks like now, as fixed
Comment 4 Beth Tibbitts CLA 2010-09-20 13:44:40 EDT
committed to HEAD too. Also removed some unused messages in LegendDialog.
I'll assume that the other icons that were in the old one no longer belong.
If they are still missing and need to be there, let me know and I'll find/fix them too.
Otherwise I'll consider this fixed.
Comment 5 Greg Watson CLA 2010-09-20 14:20:11 EDT
The legend dialog also needs to be extensible. The other icons are now resource manager specific, so the resource managers need a way of contributing the icons and description to the dialog.
Comment 6 Beth Tibbitts CLA 2010-09-20 14:52:11 EDT
I assume ResMgrs need to be able to add icons to any category (group of icons).  Do we need annotations about which ResMgr each icon comes from?
So would an extn point that allows a contribution of:
   Icon image, category to add it to, Icon name ("starting" etc), and which ResMgr it applies to 
be the right idea?
And a UI that looks something like this makes sense? (new row for an icon looks like)
ICON, Name, (ResMgr)
e.g.    (*=icon)
*    DISCOMBOBULATED   (Foo Res Mgr)

This does the simple thing of putting the Res Mgr that the status icon contribution comes from in parentheses.
Comments?

And what is a Res Mgr that would be a good first implementation?
Comment 7 Greg Watson CLA 2010-09-30 12:10:45 EDT
(In reply to comment #6)
> I assume ResMgrs need to be able to add icons to any category (group of icons).
>  Do we need annotations about which ResMgr each icon comes from?
> So would an extn point that allows a contribution of:
>    Icon image, category to add it to, Icon name ("starting" etc), and which
> ResMgr it applies to 
> be the right idea?
> And a UI that looks something like this makes sense? (new row for an icon looks
> like)
> ICON, Name, (ResMgr)
> e.g.    (*=icon)
> *    DISCOMBOBULATED   (Foo Res Mgr)
> 
> This does the simple thing of putting the Res Mgr that the status icon
> contribution comes from in parentheses.
> Comments?
> 
> And what is a Res Mgr that would be a good first implementation?

It's a little bit more complicated because of the new flexible model Randy is working on, which will allow a RM to define any number of levels (not just Machine, Node, etc.) So each RM can effectively provide it's own model with elements that are independent of other RM's. To cater for this, dialog is going to need to be RM-specific, or at least have a section for each RM. Rather than a new extension point, I would suggest modifying the existing IRuntimeModelPresentation interface (used by the runtimeModelPresentation extension point) to provide a method that creates the dialog. I would also suggest removing the button on the toolbar and instead adding a submenu to the Help menu. This submenu could have a submenu for each RM (e.g. Help>Show Legend>Open MPI, etc.)
Comment 8 Greg Watson CLA 2012-10-26 10:20:15 EDT
The legend dialog is now redundant and should be removed. The only icons still used are by the debugger, which are already described in the help documentation.
Comment 9 Beth Tibbitts CLA 2012-12-11 17:02:53 EST
Removed from Kepler