Commit 7bf6df84 authored by Brian Hall's avatar Brian Hall

Change the reply shortcut to focus the field even without a selection.

parent b78d06b7
...@@ -162,6 +162,7 @@ ...@@ -162,6 +162,7 @@
w.gl.utils.getSelectedFragment = () => { w.gl.utils.getSelectedFragment = () => {
const selection = window.getSelection(); const selection = window.getSelection();
if (selection.rangeCount === 0) return null;
const documentFragment = selection.getRangeAt(0).cloneContents(); const documentFragment = selection.getRangeAt(0).cloneContents();
if (documentFragment.textContent.length === 0) return null; if (documentFragment.textContent.length === 0) return null;
......
...@@ -39,17 +39,20 @@ ...@@ -39,17 +39,20 @@
} }
ShortcutsIssuable.prototype.replyWithSelectedText = function() { 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(); 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 the documentFragment contains more than just Markdown, don't copy as GFM.
if (documentFragment.querySelector('.md, .wiki')) return; if (documentFragment.querySelector('.md, .wiki')) return;
selected = window.gl.CopyAsGFM.nodeToGFM(documentFragment); selected = window.gl.CopyAsGFM.nodeToGFM(documentFragment);
replyField = $('.js-main-target-form #note_note');
if (selected.trim() === "") { if (selected.trim() === "") {
return; return;
} }
......
---
title: Change the reply shortcut to focus the field even without a selection.
merge_request: 8873
author: Brian Hall
...@@ -27,11 +27,19 @@ ...@@ -27,11 +27,19 @@
return this.selector = 'form.js-main-target-form textarea#note_note'; return this.selector = 'form.js-main-target-form textarea#note_note';
}); });
describe('with empty selection', function() { describe('with empty selection', function() {
return it('does nothing', function() { it('does not return an error', function() {
stubSelection('');
this.shortcut.replyWithSelectedText(); this.shortcut.replyWithSelectedText();
return expect($(this.selector).val()).toBe(''); return expect($(this.selector).val()).toBe('');
}); });
return it('triggers `input`', function() {
var focused;
focused = false;
$(this.selector).on('focus', function() {
return focused = true;
});
this.shortcut.replyWithSelectedText();
return expect(focused).toBe(true);
});
}); });
describe('with any selection', function() { describe('with any selection', function() {
beforeEach(function() { beforeEach(function() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment