@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Fix bug where the user was unable to shift navigate from 'Assigned To' to 'Title' field in Maniphest
Test Plan: go to the new task page, go to the 'Assign To' field, assign the task to someone and try to do a Shift-Tab to return to the title field
Reviewers: igoleo, btrahan
Reviewed By: btrahan
CC: aran, Korvin
Maniphest Tasks: T2760
Differential Revision: https://secure.phabricator.com/D5547
+10
-10
+1
-1
webroot/rsrc/js/javelin/core/Event.js
+1
-1
webroot/rsrc/js/javelin/core/Event.js
+9
-9
webroot/rsrc/js/javelin/lib/control/tokenizer/Tokenizer.js
+9
-9
webroot/rsrc/js/javelin/lib/control/tokenizer/Tokenizer.js
···
43
43
'change'],
44
44
45
45
properties : {
46
-
limit : null,
47
-
nextInput : null
46
+
limit : null
48
47
},
49
48
50
49
members : {
···
326
325
_onkeydown : function(e) {
327
326
var focus = this._focus;
328
327
var root = this._root;
328
+
329
+
var raw = e.getRawEvent();
330
+
if (raw.ctrlKey || raw.metaKey || raw.altKey) {
331
+
return;
332
+
}
333
+
329
334
switch (e.getSpecialKey()) {
330
335
case 'tab':
331
336
var completed = this._typeahead.submit();
332
-
if (this.getNextInput()) {
333
-
if (!completed) {
334
-
this._focus.value = '';
335
-
}
336
-
setTimeout(JX.bind(this, function() {
337
-
this.getNextInput().focus();
338
-
}), 0);
337
+
if (!completed) {
338
+
this._focus.value = '';
339
339
}
340
340
break;
341
341
case 'delete':