Bug 65665 - [ViewMgmt] [RCP] View in fixed perspective should not be minimizable
Summary: [ViewMgmt] [RCP] View in fixed perspective should not be minimizable
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P2 major (vote)
Target Milestone: 3.0 RC2   Edit
Assignee: Nick Edgar CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-06-03 22:35 EDT by Ed Burnette CLA
Modified: 2004-09-16 15:42 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ed Burnette CLA 2004-06-03 22:35:45 EDT
A fixed perspective is just that - fixed. Unchangeable, unmovable, immutable. 
It's intended to be used for RCP applications for less sophisticated users 
that might be confused if their views suddenly start closing or scurrying down 
to the bottom of the window. Therefore the minimize and maximize actions 
should not be invocable or visible anywhere, for example the view system menu 
and the view tab area.

See related bug 64836 which appears to ask for the opposite of this.
Comment 1 Nick Edgar CLA 2004-06-04 11:01:23 EDT
Do you think this should apply to editors as well?
Comment 2 Nick Edgar CLA 2004-06-04 11:09:19 EDT
I actually think this is a better story for R3.0 than what I was suggesting in
bug 64836.  

The fix is also straightforward: add the following as the first statement in
supportState(int) in ViewStack and EditorStack:

	    if (page.isFixedLayout())
	        return false;
Comment 3 Ed Burnette CLA 2004-06-04 11:22:26 EDT
Yes, as long as you just have one option on the perspective XML - fixed - then 
I think it should work for editors too. I guess you could put in more options 
like canCloseViews=true/false, canMoveEditors=true/false, but not in 3.0.

Issue: Don't you *have* to let editors be closable though, even in a fixed 
perspective? That's kind of the whole point of editors - their open/save/close 
lifecycle.
Comment 4 Nick Edgar CLA 2004-06-04 11:49:04 EDT
Sorry, I wasn't clear about the semantics for editors.  They would be closeable
but not moveable (e.g. can't reorder in same editor workbook, and can't split
editor area into multiple workbooks).

Comment 5 Nick Edgar CLA 2004-06-07 11:38:16 EDT
Michael, I think this is something we should consider for RC2.
Ed's argument is reasonable.  This change provides consistent behaviour, and
hides some bugs (e.g. Maximize is currently enabled but has no effect in a fixed
perspective).

Comment 6 Nick Edgar CLA 2004-06-08 15:32:52 EDT
I've released the changes as described above (verified by Stefan).

Still need to check the semantics of editor move in a fixed perspective.
Comment 7 Nick Edgar CLA 2004-06-09 10:31:29 EDT
Since editors are not open when the perspective is created, they will not be
marked as non-moveable or non-closeable.  They will have the same behaviour as
in a non-fixed perspective.

Marking this one as fixed.

Comment 8 Nick Edgar CLA 2004-06-09 10:44:16 EDT
Verified against browser example using binaries from I20040609-0800.