Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 119890 - [ViewMgmt] Allow control of view tab compression, when compressing use icons
Summary: [ViewMgmt] Allow control of view tab compression, when compressing use icons
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.2   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.3   Edit
Assignee: Boris Bokowski CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 125691
  Show dependency tree
 
Reported: 2005-12-08 11:35 EST by Nick Edgar CLA
Modified: 2007-06-26 13:46 EDT (History)
5 users (show)

See Also:


Attachments
Suggested patch (2.31 KB, patch)
2005-12-08 12:05 EST, Nick Edgar CLA
no flags Details | Diff
Patch to o.e.ui (1.19 KB, patch)
2005-12-08 15:08 EST, Nick Edgar CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Edgar CLA 2005-12-08 11:35:54 EST
build I20051206

As we did for editor tabs (see bug 32789), we should allow control over the amount of compression of view tabs before switching to chevron mode.

Since the set of views in a folder is much more stable than for editors, view folders should tolerate much more compression (e.g. 1 character) before switching to the chevron, so as to interfere less with users' positional memory.
Comment 1 Nick Edgar CLA 2005-12-08 12:05:36 EST
Created attachment 31388 [details]
Suggested patch

Boris and Doug, what do you think of this patch?
Comment 2 Nick Edgar CLA 2005-12-08 12:06:03 EST
See also bug 113151 for more context.
Comment 3 Nick Edgar CLA 2005-12-08 15:06:34 EST
There's a cut and paste error in the patch:
+     * Workbench preference identifier for the minimum width of editor tabs. 
should be
+     * Workbench preference identifier for the minimum width of view tabs. 
Comment 4 Nick Edgar CLA 2005-12-08 15:08:03 EST
Created attachment 31409 [details]
Patch to o.e.ui

This is needed too.
Comment 5 Boris Bokowski CLA 2005-12-09 16:16:21 EST
+1 from me.
Comment 6 Boris Bokowski CLA 2005-12-12 12:37:55 EST
Sorry - I didn't realize that this bug was assigned to me. I'll apply the patch after M4 is declared unless anyone objects. Thanks for the patch, Nick!
Comment 7 Nick Edgar CLA 2005-12-12 13:32:18 EST
Post-M4 is fine.
Comment 8 Douglas Pollock CLA 2005-12-13 09:59:51 EST
The patch looks fine.  One thought though: it would be better if the icon stayed in preference to the character.  Try compressing the tab folder a lot, and you're left with only single characters: P, V, O.  I find the icon a better visual cue.
Comment 9 Nick Edgar CLA 2005-12-13 10:22:35 EST
I agree.  Currently we only show the icon for the active tab, though, so we would have to go back on that for consistency.
Also, I think the "..." could be dropped in favour of just clipping.

Comment 10 Boris Bokowski CLA 2006-01-23 17:13:01 EST
I committed the original patch from Nick (without the cut and paste error).

I think not showing icons on inactive views was a conscious design decision. We would need to discuss this from a UI design viewpoint.

The "..." is hardwired into CTabFolder, removing it would require a change to SWT.

Released >20060123.
Comment 11 Michael Van Meekeren CLA 2006-01-26 08:35:34 EST
As for comment #9, I agree we should consider a method where we compress to the icons first.  However it was a decision not to put the icons in the background folders to help bring out the selected folder more.  Before 3.0 there were problems with users not being able to see which editor was the active editor and same for views.  So this should be considered if we put icons back in the tabs.
Comment 12 Boris Bokowski CLA 2006-01-26 10:43:07 EST
Maybe use grey versions of the icons for inactive views?
Comment 13 Boris Bokowski CLA 2006-02-14 16:29:27 EST
Reopening in order to discuss further whether we should show icons for all views.
Comment 14 Michael Van Meekeren CLA 2006-04-06 14:22:54 EDT
at this point I think we should only be discussing the text behaviour and making modifications to CTabFolder or our code that sets text bounds etc...   It's not a good idea to switch to using icons at this late stage.  We should add this to the 3.3 design item list.
Comment 15 Michael Van Meekeren CLA 2006-04-06 15:24:52 EDT
not going to icons etc... for 3.2
Comment 16 Boris Bokowski CLA 2007-06-26 13:46:02 EDT
We are now always showing icons for views.