Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 316195 - Improve "Show view" dialog
Summary: Improve "Show view" dialog
Status: RESOLVED FIXED
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.0 RC0   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-08 15:27 EDT by Oleg Besedin CLA
Modified: 2010-06-08 15:41 EDT (History)
1 user (show)

See Also:


Attachments
Patch (137.27 KB, patch)
2010-06-08 15:27 EDT, Oleg Besedin CLA
no flags Details | Diff
Illustration: adding view descriptors and "Show view" menu (6.29 KB, patch)
2010-06-08 15:31 EDT, Oleg Besedin CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.