Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 314538 - Selection service considers context layers incorrectly
Summary: Selection service considers context layers incorrectly
Status: RESOLVED FIXED
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 1.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.0 RC0   Edit
Assignee: Remy Suen CLA
QA Contact: Remy Suen CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-26 13:37 EDT by Paul Webster CLA
Modified: 2010-05-27 10:04 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Webster CLA 2010-05-26 13:37:59 EDT 
in I20100523-1612

I installed the EMF SDK and opened our /org.eclipse.e4.ui.model.workbench/model/UIElements.ecore file.  If I open the Properties view it shows the correctly selected element, but if I select a different element in the ecore editor, the Properties view is not updated.

PW
Comment 1 Remy Suen CLA 2010-05-26 14:04:04 EDT 
If you close and reopen the view, it tracks the selection of views (such as the 'Package Explorer'). It refuses to track the selection of the ecore editor though.
Comment 2 Remy Suen CLA 2010-05-26 16:12:56 EDT 
(In reply to comment #1)
> It refuses to track the selection of the ecore editor though.

Actually, it only seems to want to track the "first" view. If I have the 'Navigator' up on restart then try the 'Package Explorer' it doesn't want to work.
Comment 3 Remy Suen CLA 2010-05-27 08:15:58 EDT 
If my views are parts instead of placeholders, then it works. Not quite sure why that is affecting this at the moment.
Comment 4 Remy Suen CLA 2010-05-27 10:04:02 EDT 
Changes to selections were not being propagated upwards because the selection service was created at the window level but the scoping was done/"blocked" at the perspective level.

I've released a fix and a test.