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

Bug 316195

Summary: Improve "Show view" dialog
Product: [Eclipse Project] e4 Reporter: Oleg Besedin <ob1.eclipse>
Component: UIAssignee: Project Inbox <e4.ui-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: ob1.eclipse
Version: unspecified   
Target Milestone: 1.0 RC0   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Patch
none
Illustration: adding view descriptors and "Show view" menu none

Description Oleg Besedin CLA 2010-06-08 15:27:31 EDT
Created attachment 171464 [details]
Patch

The current e4 "Show view" dialog was a added as a quick stop-gap solution and can not be compared with 3.x analog.

The attached patch is based in 3.x code that had to be modified, in some places significantly so.

The new e4 dialog assumes that views are parts with "View" tag set and use "categoryTag:<name>" tag to specify view's category, if any. 

Differences from the 3.x base:

- we only show single level of categories
This is probably not a problem - most existing categories are one level deep.

- we don't filter activities 
Activities should be filtered by the compatibility layer?

- internally we use category names, rather then category IDs
We use tag categoryTag:<category_name>. It might be OK as long as it fits whatever translation strategy we'll pick up.

- did not register with the context help
At RCP level, there is no analog to the workbench help service at this time.

The amount of code that had to be copied (~100K) and the extent of required modifications (~2 days to make it work) underscore the need to refactor workbench plugin to make its components such as dialogs and UI jobs available at RCP level. I'll open a separate bug about that, probably to be addressed after the initial release.
Comment 1 Oleg Besedin CLA 2010-06-08 15:31:36 EDT
Created attachment 171466 [details]
Illustration: adding view descriptors and "Show view" menu

The is an illustration based on the photo demo showing a possible way to add view descriptors and a "Show view" menu.

This patch is for illustration only and is not meant to be released.
Comment 2 Oleg Besedin CLA 2010-06-08 15:41:24 EDT
Patch "Patch" applied to CVS Head. I opened bug 316196 for the future refactoring of the UI workbench plugin.