Tryton - Issues

 

Issue6104

Title keyboard shortcuts for sao
Priority feature Status chatting
Superseder Nosy List ced, perilla, pokoli, reviewbot
Type feature request Components sao
Assigned To perilla Keywords review
Reviews 32881002
View: 32881002

Created on 2016-12-10.05:44:28 by perilla, last changed by reviewbot.

Messages
review32881002 updated at https://codereview.tryton.org/32881002/#ps490001
review32881002 updated at https://codereview.tryton.org/32881002/#ps470001
review32881002 updated at https://codereview.tryton.org/32881002/#ps450001
review32881002 updated at https://codereview.tryton.org/32881002/#ps430001
review32881002 updated at https://codereview.tryton.org/32881002/#ps410001
review32881002 updated at https://codereview.tryton.org/32881002/#ps390001
review32881002 updated at https://codereview.tryton.org/32881002/#ps370001
review32881002 updated at https://codereview.tryton.org/32881002/#ps350001
review32881002 updated at https://codereview.tryton.org/32881002/#ps330001
review32881002 updated at https://codereview.tryton.org/32881002/#ps310001
review32881002 updated at https://codereview.tryton.org/32881002/#ps290001
review32881002 updated at https://codereview.tryton.org/32881002/#ps270001
review32881002 updated at https://codereview.tryton.org/32881002/#ps250001
review32881002 updated at https://codereview.tryton.org/32881002/#ps230001
review32881002 updated at https://codereview.tryton.org/32881002/#ps210001
review32881002 updated at https://codereview.tryton.org/32881002/#ps190001
review32881002 updated at https://codereview.tryton.org/32881002/#ps170001
review32881002 updated at https://codereview.tryton.org/32881002/#ps150001
review32881002 updated at https://codereview.tryton.org/32881002/#ps130001
msg30679 (view) Author: [hidden] (perilla) Date: 2016-12-12.15:31:24
Seems that chrome prevent redefine certain ctrl shortcuts

https://github.com/ccampbell/mousetrap/issues/159
https://github.com/ccampbell/mousetrap/issues/172
https://github.com/chieffancypants/angular-hotkeys/issues/157
msg30678 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2016-12-12.14:56:03
I think we should look at our shortcuts and compare with the shortcuts of popular browsers (chrome, firefox, IE and Safari). We should take the decision according to how much conflict there are and how bad they are.
msg30677 (view) Author: [hidden] (perilla) Date: 2016-12-12.14:50:47
Because ctrl keys are used by browser, but I think
that we can decide between (ctrl or alt).
msg30663 (view) Author: [hidden] (pokoli) (Tryton committer) Date: 2016-12-12.09:10:49
Why are you sugesting the alt keywork instead of ctrl key? 

I prefer the same behaviour as gtk client. 

P.S: If you plan to work on the issue please assign it to yourself.
review32881002 updated at https://codereview.tryton.org/32881002/#ps100001
review32881002 updated at https://codereview.tryton.org/32881002/#ps80001
review32881002 updated at https://codereview.tryton.org/32881002/#ps60001
review32881002 updated at https://codereview.tryton.org/32881002/#ps40001
review32881002 updated at https://codereview.tryton.org/32881002/#ps20001
New review32881002 at https://codereview.tryton.org/32881002/#ps1
msg30645 (view) Author: [hidden] (perilla) Date: 2016-12-10.05:50:03
However I think that we should use alt key instead of ctrl key
I add a initial review32881002
msg30644 (view) Author: [hidden] (perilla) Date: 2016-12-10.05:44:27
I am wondering if we add  key shortcuts on sao...
History
Date User Action Args
2017-02-20 14:37:30reviewbotsetmessages: + msg32064
2017-02-14 14:39:32reviewbotsetmessages: + msg31918
2017-02-13 17:39:27perillasettitle: key shortcuts for sao -> keyboard shortcuts for sao
2017-02-12 05:14:49reviewbotsetmessages: + msg31878
2017-02-01 18:17:32reviewbotsetmessages: + msg31648
2017-01-27 17:17:13reviewbotsetmessages: + msg31527
2017-01-27 16:48:57reviewbotsetmessages: + msg31522
2017-01-23 23:40:23reviewbotsetmessages: + msg31460
2017-01-23 22:38:59reviewbotsetmessages: + msg31458
2017-01-20 15:38:29reviewbotsetmessages: + msg31394

Showing 10 items. Show all history (warning: this could be VERY long)