mirror of
https://github.com/codex-team/editor.js
synced 2024-05-28 11:33:04 +02:00
fix(paste): delay onPaste call to be sure that block is ready
This commit is contained in:
parent
59c8d28da5
commit
46df5218ae
|
@ -23,6 +23,7 @@
|
|||
- `Improvement` - `blocks.update(id, data)` now can accept partial data object — it will update only passed properties, others will remain the same.
|
||||
- `Improvement` - `blocks.update(id, data)` now will trigger onChange with only `block-change` event.
|
||||
- `Improvement` - `blocks.update(id, data)` will return a promise with BlockAPI object of the changed block.
|
||||
- `Fix` — Some Block were be skipped on saving after pasting them as HTML
|
||||
|
||||
### 2.27.2
|
||||
|
||||
|
|
|
@ -399,7 +399,16 @@ export default class BlockManager extends Module {
|
|||
});
|
||||
|
||||
try {
|
||||
block.call(BlockToolAPI.ON_PASTE, pasteEvent);
|
||||
/**
|
||||
* We need to call onPaste after Block will be ready
|
||||
* because onPaste could change tool's root element, and we need to do that after block.watchBlockMutations() bound
|
||||
* to detect tool root element change
|
||||
*
|
||||
* @todo make this.insert() awaitable and remove requestIdleCallback
|
||||
*/
|
||||
window.requestIdleCallback(() => {
|
||||
block.call(BlockToolAPI.ON_PASTE, pasteEvent);
|
||||
});
|
||||
} catch (e) {
|
||||
_.log(`${toolName}: onPaste callback call is failed`, 'error', e);
|
||||
}
|
||||
|
|
|
@ -1,8 +1,10 @@
|
|||
import Header from '@editorjs/header';
|
||||
import Image from '@editorjs/simple-image';
|
||||
import * as _ from '../../../src/components/utils';
|
||||
import { BlockTool, BlockToolData } from '../../../types';
|
||||
import { BlockTool, BlockToolData, OutputData } from '../../../types';
|
||||
import $ from '../../../src/components/dom';
|
||||
import type EditorJS from '../../../types/index';
|
||||
|
||||
|
||||
describe('Copy pasting from Editor', function () {
|
||||
context('pasting', function () {
|
||||
|
@ -111,7 +113,7 @@ describe('Copy pasting from Editor', function () {
|
|||
tools: {
|
||||
header: Header,
|
||||
},
|
||||
});
|
||||
}).as('editorInstance');
|
||||
|
||||
cy.get('[data-cy=editorjs]')
|
||||
.get('div.ce-block')
|
||||
|
@ -121,6 +123,9 @@ describe('Copy pasting from Editor', function () {
|
|||
'text/html': '<h2>First block</h2><p>Second block</p>',
|
||||
});
|
||||
|
||||
/**
|
||||
* Check inserted blocks
|
||||
*/
|
||||
cy.get('[data-cy=editorjs]')
|
||||
.get('h2.ce-header')
|
||||
.should('contain', 'First block');
|
||||
|
@ -128,6 +133,28 @@ describe('Copy pasting from Editor', function () {
|
|||
cy.get('[data-cy=editorjs]')
|
||||
.get('div.ce-paragraph')
|
||||
.should('contain', 'Second block');
|
||||
|
||||
/**
|
||||
* Check saved data as well
|
||||
*/
|
||||
cy.get<EditorJS>('@editorInstance')
|
||||
.then(async (editor) => {
|
||||
cy.wrap<OutputData>(await editor.save())
|
||||
.then((data) => {
|
||||
/**
|
||||
* <h2> has been correctly saved
|
||||
*/
|
||||
expect(data.blocks[0].type).to.eq('header');
|
||||
expect(data.blocks[0].data.text).to.eq('First block');
|
||||
expect(data.blocks[0].data.level).to.eq(2);
|
||||
|
||||
/**
|
||||
* <p> has been correctly saved
|
||||
*/
|
||||
expect(data.blocks[1].type).to.eq('paragraph');
|
||||
expect(data.blocks[1].data.text).to.eq('Second block');
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
it('should parse pattern', function () {
|
||||
|
|
Loading…
Reference in a new issue