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

Checking to make sure there is only text in the node before removing the whole node on deleteBackward (#2476)

Browse Source 
* Checking to make sure there is only text in the node before removing the whole node.

* Make it Prettier
This commit is contained in:
ldavidpace
2018-12-04 12:08:45 -07:00
committed by Ian Storm Taylor
parent ce8bfa56a6
commit 287875b9e3
2 changed files with 38 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
packages/slate/src/commands/at-range.js
View File

@@ -312,7 +312,12 @@ Commands.deleteBackwardAtRange = (editor, range, n = 1) => {
// PERF: If the closest block is empty, remove it. This is just a shortcut, // PERF: If the closest block is empty, remove it. This is just a shortcut,
// since merging it would result in the same outcome. // since merging it would result in the same outcome.
if (document.nodes.size !== 1 && block && block.text === '') { if (
document.nodes.size !== 1 &&
block &&
block.text === '' &&
block.nodes.size === 1
) {
editor.removeNodeByKey(block.key) editor.removeNodeByKey(block.key)
return return
} }

32
packages/slate/test/commands/at-current-range/delete-backward/empty-after-multiple-void-blocks.js Normal file
View File

@@ -0,0 +1,32 @@
/** @jsx h */
import h from '../../../helpers/h'
export default function(editor) {
editor.deleteBackward()
}
export const input = (
<value>
<document>
<paragraph>
<emoji />
<emoji />
<cursor />
</paragraph>
<paragraph />
</document>
</value>
)
export const output = (
<value>
<document>
<paragraph>
<emoji />
<cursor />
</paragraph>
<paragraph />
</document>
</value>
)