Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 332137

Summary: Conflicting keyboard shortcut: CTRL+ALT+P (Prepare Changelog vs. Prepare Changelog in an editor)?
Product: [Tools] Linux Tools Reporter: Severin Gehwolf <sgehwolf>
Component: ChangeLogAssignee: Andrew Ferrazzutti <af_0_af>
Status: RESOLVED INVALID QA Contact:
Severity: normal    
Priority: P3 CC: akurtakov, jjohnstn, linux.changelog-inbox, overholt
Version: 0.7.0   
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Whiteboard: stalebug

Description Severin Gehwolf CLA 2010-12-08 10:19:11 EST
I'm seeing this in my error log. Should we care?

Warning
Wed Dec 08 10:07:23 EST 2010
A conflict occurred for ALT+CTRL+P:Binding(ALT+CTRL+P,
ParameterizedCommand(Command(org.eclipse.linuxtools.changelog.core.preparechangelog2,Prepare Changelog In Editor,
Prepares ChangeLog in an editor,
Category(org.eclipse.linuxtools.changelog,Changelog,Changelog key bindings,true),
ActionDelegateHandlerProxy(null,org.eclipse.linuxtools.changelog.core.actions.PrepareChangelogKeyHandler),
,,true),null),
org.eclipse.ui.defaultAcceleratorConfiguration,
org.eclipse.ui.textEditorScope,,,system)
Binding(ALT+CTRL+P,
ParameterizedCommand(Command(org.eclipse.linuxtools.changelog.core.preparechangelog2,Prepare Changelog In Editor,
Prepares ChangeLog in an editor,
Category(org.eclipse.linuxtools.changelog,Changelog,Changelog key bindings,true),
ActionDelegateHandlerProxy(null,org.eclipse.linuxtools.changelog.core.actions.PrepareChangelogKeyHandler),
,,true),null),
org.eclipse.ui.defaultAcceleratorConfiguration,
org.eclipse.compare.compareEditorScope,,,system)
Binding(ALT+CTRL+P,
ParameterizedCommand(Command(org.eclipse.linuxtools.changelog.core.preparechangelog,Prepare Changelog,
Prepares Changelog,
Category(org.eclipse.linuxtools.changelog,Changelog,Changelog key bindings,true),
ActionDelegateHandlerProxy(null,org.eclipse.linuxtools.changelog.core.actions.PrepareChangelogKeyHandler),
,,true),null),
org.eclipse.ui.defaultAcceleratorConfiguration,
org.eclipse.ui.contexts.window,,,system)
Comment 1 Andrew Overholt CLA 2011-01-10 10:10:38 EST
It seems like a bug to me :)
Comment 2 Andrew Overholt CLA 2011-05-30 11:33:37 EDT
When was this fixed, Jeff?  0.8?
Comment 3 Alexander Kurtakov CLA 2012-04-19 03:49:11 EDT
Is this bug really fixed ?
Should we close it?
Comment 4 Eclipse Genie CLA 2014-06-11 17:45:56 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.
Comment 5 Alexander Kurtakov CLA 2014-06-13 09:35:53 EDT
Andrew, would you please look into this one.
Comment 6 Andrew Ferrazzutti CLA 2014-06-16 12:04:31 EDT
I can confirm that this error still happens, and that there's also apparently a conflict with the ALT+CTRL+C binding as well. I'll look into it.
Comment 7 Jeff Johnston CLA 2014-06-16 12:47:33 EDT
(In reply to Andrew Ferrazzutti from comment #6)
> I can confirm that this error still happens, and that there's also
> apparently a conflict with the ALT+CTRL+C binding as well. I'll look into it.


There was another bug for this issue which I proposed a solution and didn't receive any comments:

https://bugs.eclipse.org/bugs/show_bug.cgi?id=296587
Comment 8 Andrew Ferrazzutti CLA 2015-05-06 14:53:56 EDT
At the time of this writing, the keybinding conflicts still exist, but they don't cause any problems at user-level. Since each of the "conflicting" keybindings are each active in a different, non-conflicting context, there is no real conflict to speak of.

For instance, the ALT+CTRL+P keybindings that trigger the "Prepare Changelog" command are each active in three non-conflicting contexts: Window, Text Editor, and Compare Editor. I tested using that keybinding in those contexts, and as far as I could tell, they never conflict with each other.

The ALT+CTRL+C keybinding has a similar situation: the same keybinding is used for Subversive SVN's "Prepare Commit" command, but it's only active in Windows whereas Changelog's keybinding is active in Text/Compare Editors. Due to this, they don't conflict with each other in practice.

Finally, when looking through my Eclipse's Error Log, I found some other keybinding conflicts from other plugins. Trying to use any one of them causes the Key Assist popup to appear, which lists all of the commands provided by that keybinding and lets you choose the one you want to execute. It's the same popup that appears when you input an incomplete two-phase key shortcut, like Shift+Alt+X or Shift+Alt+Q. This also means that we don't need to worry about future plugins that will potentially use the same keybindings as Changelog (a point that was made in #296587). I wonder if Eclipse had this feature when this bug was first reported, since it's a good solution to the general issue of key conflicts.

Considering all this, I'm marking this as resolved. If anyone still has concerns about it, we can discuss it in the comments, but I don't see much of a reason to leave this as an open bug.