diff options
author | Brian Hall <brian@hack.design> | 2017-01-30 22:12:31 -0600 |
---|---|---|
committer | Brian Hall <brian@hack.design> | 2017-01-31 12:28:49 -0600 |
commit | 7bf6df8463c4f8871682f385e9368d169b4ffecf (patch) | |
tree | 5848818ea63a6f69b19b5d4b5b49433b6229d137 /app/assets/javascripts/shortcuts_issuable.js | |
parent | b78d06b78143b16dccc5d5afaa8796473b68bea1 (diff) | |
download | gitlab-ce-7bf6df8463c4f8871682f385e9368d169b4ffecf.tar.gz |
Change the reply shortcut to focus the field even without a selection.
Diffstat (limited to 'app/assets/javascripts/shortcuts_issuable.js')
-rw-r--r-- | app/assets/javascripts/shortcuts_issuable.js | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/app/assets/javascripts/shortcuts_issuable.js b/app/assets/javascripts/shortcuts_issuable.js index 4ef516af8c8..4dcc5ebe28f 100644 --- a/app/assets/javascripts/shortcuts_issuable.js +++ b/app/assets/javascripts/shortcuts_issuable.js @@ -39,17 +39,20 @@ } ShortcutsIssuable.prototype.replyWithSelectedText = function() { - var quote, replyField, documentFragment, selected, separator; + var quote, documentFragment, selected, separator; + var replyField = $('.js-main-target-form #note_note'); documentFragment = window.gl.utils.getSelectedFragment(); - if (!documentFragment) return; + if (!documentFragment) { + replyField.focus(); + return; + } // If the documentFragment contains more than just Markdown, don't copy as GFM. if (documentFragment.querySelector('.md, .wiki')) return; selected = window.gl.CopyAsGFM.nodeToGFM(documentFragment); - replyField = $('.js-main-target-form #note_note'); if (selected.trim() === "") { return; } |