Przeglądaj źródła

Merge branch 'master' into i/6114

Piotrek Koszuliński 5 lat temu
rodzic
commit
cb3492485a

+ 2 - 0
packages/ckeditor5-table/src/tablecellproperties/commands/tablecellpropertycommand.js

@@ -114,6 +114,8 @@ export default class TableCellPropertyCommand extends Command {
 }
 
 // Returns all selected table cells.
+// The implementation of this function is incorrect as it may return a single cell twice.
+// See https://github.com/ckeditor/ckeditor5/issues/6358.
 function getSelectedTableCells( model ) {
 	const selection = model.document.selection;
 

+ 1 - 0
packages/ckeditor5-table/src/tableselection.js

@@ -229,6 +229,7 @@ export default class TableSelection extends Plugin {
 	 * @returns {Iterable.<module:engine/model/element~Element>}
 	 */
 	* getSelectedTableCells() {
+		// TODO: this function should be removed. See https://github.com/ckeditor/ckeditor5/issues/6358
 		if ( !this.hasMultiCellSelection ) {
 			return;
 		}

+ 1 - 0
packages/ckeditor5-table/src/ui/utils.js

@@ -81,6 +81,7 @@ export function getBalloonTablePositionData( editor ) {
  * @returns {module:utils/dom/position~Options}
  */
 export function getBalloonCellPositionData( editor ) {
+	// This is a bit naive. See https://github.com/ckeditor/ckeditor5/issues/6357.
 	const modelTableCell = getTableCellAtPosition( editor.model.document.selection.getFirstPosition() );
 	const viewTableCell = editor.editing.mapper.toViewElement( modelTableCell );