Bug #534

CTRL+Q shortcut collision in theide

Added by Zbigniew Rebacz over 10 years ago. Updated over 10 years ago.

Status:ApprovedStart date:10/05/2013
Priority:NormalDue date:
Assignee:Zbigniew Rebacz% Done:

100%

Category:IDESpent time:-
Target version:-

Description

It seems that assigning a CTRL+Q shortcut to "exit" resulted collision with "search symbol". Personally, I think we should replace "search symbol" shortcut key with CTRL+W. It is nice combination, because it is free, fast and similar to previous one.

P.S.
Moreover I enclose small cosmetic "navigator" fix.

Navigator.cpp Magnifier (3.03 KB) Zbigniew Rebacz, 10/05/2013 01:40 PM

ide.key - ide.key without collisions (4.25 KB) Zbigniew Rebacz, 10/08/2013 03:28 AM

History

#1 Updated by Zbigniew Rebacz over 10 years ago

  • Status changed from New to Ready for QA

#2 Updated by Jan DolinĂ¡r over 10 years ago

It seems that assigning a CTRL+Q shortcut to "exit" resulted collision with "search symbol". Personally, I think we should replace "search symbol" shortcut key with CTRL+W. It is nice combination, because it is free, fast and similar to previous one.

I agree with Ctrl+Q for exit, but Ctrl+W is most commonly used for closing tab/file (currently Ctrl+U in TheIDE). If we correct both the closing shortcuts, it would leave the Ctrl+U free so it could be used for search symbol (unless there is some better option). What do you think?

#3 Updated by Miroslav Fidler over 10 years ago

  • Status changed from Ready for QA to Approved

#4 Updated by Zbigniew Rebacz over 10 years ago

  • Status changed from Approved to Rejected

Jan, I fully support your position, but there is one big problem with current upp version. Now we have collision with CTRL+U. We need to modify following lines as well (ide.key):
KEY
KEY

Insted of:

KEY
KEY

This solution should break collision chain.

#5 Updated by Zbigniew Rebacz over 10 years ago

I don't know what happend with my code. I put it in a file form.

#6 Updated by Zbigniew Rebacz over 10 years ago

  • File deleted (ide.key)

#7 Updated by Zbigniew Rebacz over 10 years ago

  • Status changed from Rejected to Ready for QA

#8 Updated by Miroslav Fidler over 10 years ago

Done...

#9 Updated by Zbigniew Rebacz over 10 years ago

  • Status changed from Ready for QA to Approved

Also available in: Atom PDF