Bug 482647

Summary: MPerspective should also extends MBindings
Product: [Eclipse Project] Platform Reporter: Lars Vogel <Lars.Vogel>
Component: UIAssignee: Lars Vogel <Lars.Vogel>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: jhelming, Lars.Vogel, loskutov, simon.scholz, tom.schindl
Version: 4.6Keywords: noteworthy
Target Milestone: 4.6 M6   
Hardware: PC   
OS: Windows NT   
See Also: https://git.eclipse.org/r/67340
https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=9bdf73b60f99fad168d23aa6b712b361462b12b8
Whiteboard:

Description Lars Vogel CLA 2015-11-20 04:39:33 EST
I think it makes sense to allow the definition of key bindings for a perspective.
Comment 1 Lars Vogel CLA 2015-11-20 04:40:37 EST
Anyone who disagrees with such a change?
Comment 2 Andrey Loskutov CLA 2015-11-20 05:04:50 EST
Lars, do you mean that e.g. "C++ perspective" should be able to define binding so that user can switch to the perspective, or do you mean you want a new scope for the key bindings?
Comment 3 Lars Vogel CLA 2015-11-20 05:40:46 EST
(In reply to Andrey Loskutov from comment #2)
> Lars, do you mean that e.g. "C++ perspective" should be able to define
> binding so that user can switch to the perspective, or do you mean you want
> a new scope for the key bindings?

Nether. :-)

In e4 RCP you can define keybindings and assign them directly to applications, windows and views / editors. The framework activates them automatically if the window / view etc. is activated. For perspectives (in e4 RCP) this attribute is not available, hence the customer needs to define an event listener which does this work. This feels inconsistent with the rest of the e4 application model. And I have a customer which would like to use this.
Comment 4 Lars Vogel CLA 2015-11-20 05:42:52 EST
Suggested change is similar to "Perspective becomes a handler container" from https://www.eclipse.org/eclipse/news/4.6/M2/
Comment 5 Jonas Helming CLA 2015-11-20 08:20:33 EST
+1
Comment 6 Lars Vogel CLA 2016-01-25 04:56:35 EST
Mass move to M6