mirror/slate
1
0
Fork 0
mirror of https://github.com/ianstormtaylor/slate.git synced 2025-08-18 21:21:21 +02:00
Code Issues Releases Wiki Activity

Be inclusive for selection in insertText

Browse Source
This commit is contained in:
Samy Pessé
2016-10-26 14:07:26 +02:00
parent 9f5807e008
commit 99e9a77639
3 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/transforms/apply-operation.js
View File

@@ -115,10 +115,10 @@ function insertText(state, operation) {
document = document.updateDescendant(node) document = document.updateDescendant(node)
// Update the selection // Update the selection
if (startKey == node.key && startOffset > offset) { if (startKey == node.key && startOffset >= offset) {
selection = selection.moveStartOffset(text.length) selection = selection.moveStartOffset(text.length)
} }
if (endKey == node.key && endOffset > offset) { if (endKey == node.key && endOffset >= offset) {
selection = selection.moveEndOffset(text.length) selection = selection.moveEndOffset(text.length)
} }

4
test/transforms/fixtures/by-key/insert-text-by-key/selection-end/index.js
View File

@@ -21,7 +21,9 @@ export default function (state) {
assert.deepEqual( assert.deepEqual(
next.selection.toJS(), next.selection.toJS(),
nextSelection.toJS() nextSelection.merge({
focusOffset: 6
}).toJS()
) )
return next return next

1
test/transforms/fixtures/by-key/insert-text-by-key/selection-start/index.js
View File

@@ -22,6 +22,7 @@ export default function (state) {
assert.deepEqual( assert.deepEqual(
next.selection.toJS(), next.selection.toJS(),
nextSelection.merge({ nextSelection.merge({
anchorOffset: 4,
focusOffset: 6 focusOffset: 6
}).toJS() }).toJS()
) )