Bug 482647 - MPerspective should also extends MBindings
Summary: MPerspective should also extends MBindings
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.6   Edit
Hardware: PC Windows NT
: P3 normal (vote)
Target Milestone: 4.6 M6   Edit
Assignee: Lars Vogel CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on:
Blocks:
 
Reported: 2015-11-20 04:39 EST by Lars Vogel CLA
Modified: 2016-03-01 02:28 EST (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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