Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 363648 - Keyboard shortcut Ctrl+# does not bring up Commit dialog while editing code
Summary: Keyboard shortcut Ctrl+# does not bring up Commit dialog while editing code
Status: RESOLVED FIXED
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: 1.1   Edit
Hardware: All All
: P3 normal with 1 vote (vote)
Target Milestone: 3.0   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 401411
Blocks:
  Show dependency tree
 
Reported: 2011-11-12 13:40 EST by jason CLA
Modified: 2014-02-03 10:15 EST (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description jason CLA 2011-11-12 13:40:10 EST
Build Identifier: 20110916-0149

It is necessary for me to change the default key binding for the Commit in EGit as command+(shift+3) is the os shortcut for taking a screen shot.

I picked command+k modeled after Pycharm. When I use the shortcut no dialog comes up as I would expect it to when right clicking Team->commit.

There are no conflicts and command+k works in other applications (Iterm2 and Clearing the console)

Here is a pretty screenshot of my key bindings: http://i.imgur.com/EzolB.png

Reproducible: Always

Steps to Reproduce:
1. push keyboard shortcut for Commit

Nothing happens.
Comment 1 Remy Suen CLA 2011-11-15 08:21:29 EST
Jason, please put your comments and findings in bug 356647.

*** This bug has been marked as a duplicate of bug 356647 ***
Comment 2 Markus Keller CLA 2012-10-23 15:02:16 EDT
Not a dup.

The problem here is that the Commit... shortcut is not handled unless the Git action set group is active in the current perspective. A workaround is to do "Window > Customize Perspective...", go to the "Command Groups Availability" tab, and make sure "Git" is checked in the left column.

The "Customize Perspective" dialog is still very broken in 4.2 and 4.3 (bug 378849), so have to close the dialog and then reopen it if you want hide the "Git" toolbar again.
Comment 3 Matthias Sohn CLA 2013-02-19 17:27:02 EST
(In reply to comment #2)
> Not a dup.
> 
> The problem here is that the Commit... shortcut is not handled unless the
> Git action set group is active in the current perspective. A workaround is
> to do "Window > Customize Perspective...", go to the "Command Groups
> Availability" tab, and make sure "Git" is checked in the left column.
> 
> The "Customize Perspective" dialog is still very broken in 4.2 and 4.3 (bug
> 378849), so have to close the dialog and then reopen it if you want hide the
> "Git" toolbar again.

so should this bug be moved to platform ?
Comment 4 Markus Keller CLA 2013-02-22 07:03:31 EST
I think the problem from comment 2 has been fixed in the meantime.

But I found that the fix for bug 356647 was bad, since it missed to declare the platform in the deletion marker, so it effectively deletes the key binding on all platforms. Robin R did notice this, but unfortunately, I convinced him that the fix was OK. Sorry about that.

Fix: https://git.eclipse.org/r/#/c/10586/

This is a regression compared to 2.2. It would be good to add this fix to 2.3.1.
Comment 5 Matthias Sohn CLA 2013-02-23 17:20:37 EST
merged as da62d62de46736242b4ef0411f3450514f9b40c8
Comment 6 Dani Megert CLA 2013-02-25 03:53:58 EST
Verified on Windows 7 with 2.4.0.201302242046 that it works.

Someone on a Mac should also verify it.
Comment 7 Piotr Findeisen CLA 2014-02-03 10:15:32 EST
I am using Eclipse Kepler sr1 with Egit 3.2.0-something on Linux and still have this problem.

I confirm that adding git in "Customize perspective" > "Command groups availability" is a workaround, but it clutters GUI with lots of icons I will never touch.

The default shortcut (ctrl+shift+3) works perfectly in e.g. Package Explorer or Git Repositories views, but not in Java editor. 

I checked preferences>keys but could not find anything conflicting. There is "Shift+Ctrl+3" shortcut for javascript debugging (i edit java, not javascript), but removing the shortcut does not help.

Is it something broken in my eclipse config?