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

Fix activeMarks at the beginning of a line (#1434)

Browse Source 
If you move to the beginning of a text node, and the previous text
node has marks, `marks` is set but `activeMarks` is not. This was
caused by `getActiveMarksAtRangeAsArray` looking for a function that
didn't exist. With this change, `activeMarks` is brought in line with
how `marks` works for collapsed selections.
This commit is contained in:
Justin Weiss
2017-12-02 12:35:16 -08:00
committed by Ian Storm Taylor
parent 7032339ca2
commit 105b384ccb
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
packages/slate/src/models/node.js
View File

@@ -1082,8 +1082,8 @@ class Node {
// If the range is collapsed at the start of the node, check the previous. // If the range is collapsed at the start of the node, check the previous.
if (range.isCollapsed && startOffset == 0) { if (range.isCollapsed && startOffset == 0) {
const previous = this.getPreviousText(startKey) const previous = this.getPreviousText(startKey)
if (!previous || !previous.length) return [] if (!previous || previous.text.length == 0) return []
const char = previous.characters.get(previous.length - 1) const char = previous.characters.get(previous.text.length - 1)
return char.marks.toArray() return char.marks.toArray()
} }