8
0
Просмотр исходного кода

Remove Position, Range, Selection imports from engine/model.

Maciej Gołaszewski 7 лет назад
Родитель
Сommit
7c62805c45

+ 2 - 2
packages/ckeditor5-media-embed/package.json

@@ -10,13 +10,13 @@
     "ckeditor5-plugin"
   ],
   "dependencies": {
-    "@ckeditor/ckeditor5-core": "^11.0.1",
     "@ckeditor/ckeditor5-clipboard": "^10.0.3",
+    "@ckeditor/ckeditor5-core": "^11.0.1",
     "@ckeditor/ckeditor5-engine": "^11.0.0",
     "@ckeditor/ckeditor5-enter": "^10.1.2",
     "@ckeditor/ckeditor5-ui": "^11.1.0",
-    "@ckeditor/ckeditor5-utils": "^11.0.0",
     "@ckeditor/ckeditor5-undo": "^10.0.3",
+    "@ckeditor/ckeditor5-utils": "^11.0.0",
     "@ckeditor/ckeditor5-widget": "^10.3.0"
   },
   "devDependencies": {

+ 5 - 5
packages/ckeditor5-media-embed/src/automediaembed.js

@@ -12,7 +12,6 @@ import Plugin from '@ckeditor/ckeditor5-core/src/plugin';
 import Clipboard from '@ckeditor/ckeditor5-clipboard/src/clipboard';
 import LiveRange from '@ckeditor/ckeditor5-engine/src/model/liverange';
 import LivePosition from '@ckeditor/ckeditor5-engine/src/model/liveposition';
-import TreeWalker from '@ckeditor/ckeditor5-engine/src/model/treewalker';
 import Undo from '@ckeditor/ckeditor5-undo/src/undo';
 import global from '@ckeditor/ckeditor5-utils/src/dom/global';
 
@@ -77,10 +76,10 @@ export default class AutoMediaEmbed extends Plugin {
 		this.listenTo( editor.plugins.get( Clipboard ), 'inputTransformation', () => {
 			const firstRange = modelDocument.selection.getFirstRange();
 
-			const leftLivePosition = LivePosition.createFromPosition( firstRange.start );
+			const leftLivePosition = LivePosition.fromPosition( firstRange.start );
 			leftLivePosition.stickiness = 'toPrevious';
 
-			const rightLivePosition = LivePosition.createFromPosition( firstRange.end );
+			const rightLivePosition = LivePosition.fromPosition( firstRange.end );
 			rightLivePosition.stickiness = 'toNext';
 
 			modelDocument.once( 'change:data', () => {
@@ -113,8 +112,9 @@ export default class AutoMediaEmbed extends Plugin {
 	_embedMediaBetweenPositions( leftPosition, rightPosition ) {
 		const editor = this.editor;
 		const mediaRegistry = editor.plugins.get( MediaEmbedEditing ).registry;
+		// TODO: Use marker instead of LiveRange & LivePositions.
 		const urlRange = new LiveRange( leftPosition, rightPosition );
-		const walker = new TreeWalker( { boundaries: urlRange, ignoreElementEnd: true } );
+		const walker = urlRange.getWalker( { ignoreElementEnd: true } );
 
 		let url = '';
 
@@ -137,7 +137,7 @@ export default class AutoMediaEmbed extends Plugin {
 		}
 
 		// Position won't be available in the `setTimeout` function so let's clone it.
-		this._positionToInsert = LivePosition.createFromPosition( leftPosition );
+		this._positionToInsert = LivePosition.fromPosition( leftPosition );
 
 		// This action mustn't be executed if undo was called between pasting and auto-embedding.
 		this._timeoutId = global.window.setTimeout( () => {

+ 1 - 1
packages/ckeditor5-media-embed/src/mediaembedcommand.js

@@ -62,7 +62,7 @@ export default class MediaEmbedCommand extends Command {
 				writer.setAttribute( 'url', url, selectedMedia );
 			} );
 		} else {
-			const insertPosition = findOptimalInsertionPosition( selection );
+			const insertPosition = findOptimalInsertionPosition( selection, model );
 
 			model.change( writer => {
 				const mediaElement = writer.createElement( 'media', { url } );

+ 5 - 6
packages/ckeditor5-media-embed/tests/automediaembed.js

@@ -15,8 +15,7 @@ import List from '@ckeditor/ckeditor5-list/src/list';
 import Bold from '@ckeditor/ckeditor5-basic-styles/src/bold';
 import Undo from '@ckeditor/ckeditor5-undo/src/undo';
 import Typing from '@ckeditor/ckeditor5-typing/src/typing';
-import ModelRange from '@ckeditor/ckeditor5-engine/src/model/range';
-import ModelPosition from '@ckeditor/ckeditor5-engine/src/model/position';
+
 import global from '@ckeditor/ckeditor5-utils/src/dom/global';
 import { getData, setData } from '@ckeditor/ckeditor5-engine/src/dev-utils/model';
 
@@ -420,7 +419,7 @@ describe( 'AutoMediaEmbed - integration', () => {
 
 				setTimeout( () => {
 					editor.model.enqueueChange( 'transparent', writer => {
-						writer.insertText( characters[ i ], new ModelPosition( rootEl, [ 0, i ] ) );
+						writer.insertText( characters[ i ], writer.createPositionFromPath( rootEl, [ 0, i ] ) );
 					} );
 				}, i * 5 );
 			}
@@ -462,7 +461,7 @@ describe( 'AutoMediaEmbed - integration', () => {
 			pasteHtml( editor, 'https://www.youtube.com/watch?v=H08tGjXNHO4' );
 
 			editor.model.enqueueChange( 'transparent', writer => {
-				writer.remove( ModelRange.createOn( editor.model.document.getRoot().getChild( 1 ) ) );
+				writer.remove( writer.createRangeOn( editor.model.document.getRoot().getChild( 1 ) ) );
 			} );
 
 			setTimeout( () => {
@@ -481,7 +480,7 @@ describe( 'AutoMediaEmbed - integration', () => {
 
 			editor.model.enqueueChange( 'transparent', writer => {
 				const paragraph = writer.createElement( 'paragraph' );
-				writer.insert( paragraph, ModelPosition.createAfter( editor.model.document.getRoot().getChild( 0 ) ) );
+				writer.insert( paragraph, writer.createPositionAfter( editor.model.document.getRoot().getChild( 0 ) ) );
 				writer.setSelection( paragraph, 'in' );
 			} );
 
@@ -512,7 +511,7 @@ describe( 'AutoMediaEmbed - integration', () => {
 
 			editor.model.enqueueChange( 'transparent', writer => {
 				const paragraph = writer.createElement( 'paragraph' );
-				writer.insert( paragraph, ModelPosition.createAfter( editor.model.document.getRoot().getChild( 1 ) ) );
+				writer.insert( paragraph, writer.createPositionAfter( editor.model.document.getRoot().getChild( 1 ) ) );
 				writer.setSelection( paragraph, 'in' );
 			} );