Bug 80753 - [Actions] use new command story for CVS keybindings, actions
Summary: [Actions] use new command story for CVS keybindings, actions
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Team (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.1 M6   Edit
Assignee: Platform Team Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-12-10 22:57 EST by Jean-Michel Lemieux CLA
Modified: 2005-01-05 16:16 EST (History)
0 users

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 Jean-Michel Lemieux CLA 2004-12-10 22:57:51 EST 
With the current keybinding support for CVS, if the CVS action set is not
enabled keybindings for CVS actions will not work. But this is counter-intuitive
since the popup menus in Team, and the Synchronize View will display the bindings.

The real solution is to support putting commands into the action set and then
registering default command handlers for CVS actions. When a part is activated,
it can override with it's own handler. If there are no parts that contribute a
handler, the default is used.
Comment 1 Jean-Michel Lemieux CLA 2004-12-11 14:02:42 EST 
Actually fixed this too. Trick is to add handler submissions to act as the
default handler for the keybindings, then ensure to register actions with
keybinding service and voila. Will try an release for M4, but may wait until after.
Comment 2 Jean-Michel Lemieux CLA 2005-01-05 16:16:45 EST 
released to M4 with some minor problems already captured in existing bugs.