Bug 275028 - [breadcrumb] Rename "View Mode" sub-menu and other breadcrumb related actions in Debug view
Summary: [breadcrumb] Rename "View Mode" sub-menu and other breadcrumb related actions...
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 3.5   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 3.5 RC1   Edit
Assignee: Samantha Chan CLA
QA Contact:
URL:
Whiteboard:
Keywords: polish
Depends on:
Blocks:
 
Reported: 2009-05-05 13:22 EDT by Pawel Piech CLA
Modified: 2009-05-06 16:51 EDT (History)
2 users (show)

See Also:
darin.eclipse: review+


Attachments
Patch with fix. (2.25 KB, patch)
2009-05-06 12:24 EDT, Pawel Piech CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pawel Piech CLA 2009-05-05 13:22:30 EDT
I've thought about the "View Mode" menu in Debug view and I think making the following changes would be helpful:

1) Rename the "View Mode" sub-men to "Layout".  This will make it consistent with other debugger views and make the name more descriptive.  

2) Rename "Full" and "Compact" actions to "Tree" and "Breadcrumb" respectively.  While the former names may be considered more descriptive, the latter names are more direct and presumably users are already familiar with their meaning.  JDT already uses the term "Breadcrumb" in the toolbar, and the term "Tree" is a pretty universal concept in GUIs.

3) Rename the mysterious "Auto-Expand Compact Selector" to "Auto-expand in Breadcrumb Drop-down".  It's still long, but using "Breadcrumb" and "Drop-down" should make the description more direct and easy to understand... I hope.
Comment 1 Darin Wright CLA 2009-05-05 15:24:17 EDT
Sam, do you approve of this for RC1? I think these are good polish items.
Comment 2 Samantha Chan CLA 2009-05-06 10:01:30 EDT
+1 from me to try to address this for RC1. 

1) I like the idea of renaming View Mode to Layout to make it consistent with other views.

2) I am also ok with using breadcrumb and tree in the label, although I find Full and Compact more intuitive.  I don't think many people understand what breadcrumb means, whether it's from JDT or from us.

3)  I think the label for the "Auto-Expand" action is too long.  This issue can be more serious in translated products.  The translations can be much longer then the original string.  What about "Auto-Expand Breadcrumb"?  Would that be too confusing?

I am wondering if it's too heavy weight to put this in a preference page, and have an action in the drop down that links to the preference page.  Since breadcrumb can be enabled by resizing, I don't think many people will need to use the actions frequently.  Having it in the preference page gives us more freedom on how to explain these concepts, and come up with more meaningful labels?
Comment 3 Pawel Piech CLA 2009-05-06 12:24:12 EDT
Created attachment 134642 [details]
Patch with fix.

(In reply to comment #2)

This patch (just the properties file) renames contains the changes including the "Auto-Expand Breadcrumb" label.  I also agree that the name is too long and I'm starting to think that no matter how long the label it's still going to be confusing.  It would be easier to explain it if the action was closer to the affected UI, like in the context menu in the drop-down itself, but that would e a lot of work just for a single action.  In the end I suppose that we'll need to rely on help to clarify what this thing is for... and that's what I'm working on now.

> I am wondering if it's too heavy weight to put this in a preference page, and
> have an action in the drop down that links to the preference page.  Since
> breadcrumb can be enabled by resizing, I don't think many people will need to
> use the actions frequently.  Having it in the preference page gives us more
> freedom on how to explain these concepts, and come up with more meaningful
> labels?
I suspect that there is some guideline against putting view-specific preferences in the preference pages.  Besides, if it's even further removed from the affected UI, it would be harder to explain still.
Comment 4 Pawel Piech CLA 2009-05-06 15:06:52 EDT
With the two approvals, I committed the change.  Samantha, please verify.
Comment 5 Pawel Piech CLA 2009-05-06 15:08:20 EDT
I see, I guess I'm supposed to use the review flag in addition to assigning the bug for verify.  Darin, please review.
Comment 6 Darin Wright CLA 2009-05-06 16:42:46 EDT
Verified.
Comment 7 Darin Wright CLA 2009-05-06 16:51:51 EDT
Verified.