Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 336171 - [client] generalize eWebBorderContainer
Summary: [client] generalize eWebBorderContainer
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Client (show other bugs)
Version: 0.2   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 0.2   Edit
Assignee: Susan McCourt CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-02 17:20 EST by Susan McCourt CLA
Modified: 2011-09-01 11:41 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Susan McCourt CLA 2011-02-02 17:20:31 EST
I was hoping to quickly steal the toggling splitter in the eWebBorderContainer for the navigators.  Unfortunately it currently assumes that it lives in an editorContainer, and it stores its settings to a particular location, thus it can't be reused as is.  I started hacking it and realized I was starting to go off on a tangent.

- client should pass in the DOM ids of any important panes (right now it is using "editorContainer")
- client should specify what the setting name should be for storing the location info (so different pages could use different settings)
- could we take the split management code out of editorContainer and put it in the splitter?  So that pages using the splitter don't have to get involved in the size lookups, etc.?
Comment 1 Susan McCourt CLA 2011-02-20 19:36:00 EST
It'd be nice to be able to close favorites from the navigators.
Libing, is this something you could easily do or should I give it a shot?
Comment 2 Susan McCourt CLA 2011-02-21 11:32:45 EST
I'm going to be doing some refactoring in editorContainer, so I'll take this as a to-do.
Comment 3 Susan McCourt CLA 2011-03-30 19:13:39 EDT
Fixed.
Moved what used to be in editorContainer (and temporarily in the coding glue splitterMgr) to the eWebBorderContainer.  

> - client should pass in the DOM ids of any important panes (right now it is
> using "editorContainer")

Since the code that used to look up the panes by id is now in the border container, it knows which panes are where and we don't need to pass in anything.

> - client should specify what the setting name should be for storing the
> location info (so different pages could use different settings)

This is no longer a problem. It was a general issue with border container.  It was fixed when we fixed bug 338764.  See bug 338764 comment 6.

> - could we take the split management code out of editorContainer and put it in
> the splitter?  So that pages using the splitter don't have to get involved in
> the size lookups, etc.?

Done