@ckeditor/ckeditor5-table 46.1.1 → 47.0.0-alpha.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ckeditor/ckeditor5-table",
-  "version": "46.1.1",
+  "version": "47.0.0-alpha.0",
   "description": "Table feature for CKEditor 5.",
   "keywords": [
     "ckeditor",
@@ -13,14 +13,14 @@
   "type": "module",
   "main": "src/index.js",
   "dependencies": {
-    "ckeditor5": "46.1.1",
-    "@ckeditor/ckeditor5-clipboard": "46.1.1",
-    "@ckeditor/ckeditor5-core": "46.1.1",
-    "@ckeditor/ckeditor5-engine": "46.1.1",
-    "@ckeditor/ckeditor5-icons": "46.1.1",
-    "@ckeditor/ckeditor5-ui": "46.1.1",
-    "@ckeditor/ckeditor5-utils": "46.1.1",
-    "@ckeditor/ckeditor5-widget": "46.1.1",
+    "ckeditor5": "47.0.0-alpha.0",
+    "@ckeditor/ckeditor5-clipboard": "47.0.0-alpha.0",
+    "@ckeditor/ckeditor5-core": "47.0.0-alpha.0",
+    "@ckeditor/ckeditor5-engine": "47.0.0-alpha.0",
+    "@ckeditor/ckeditor5-icons": "47.0.0-alpha.0",
+    "@ckeditor/ckeditor5-ui": "47.0.0-alpha.0",
+    "@ckeditor/ckeditor5-utils": "47.0.0-alpha.0",
+    "@ckeditor/ckeditor5-widget": "47.0.0-alpha.0",
     "es-toolkit": "1.39.5"
   },
   "author": "CKSource (http://cksource.com/)",

package/src/tablecellproperties/tablecellpropertiesediting.d.ts

@@ -32,10 +32,19 @@ export declare class TableCellPropertiesEditing extends Plugin {
      * @inheritDoc
      */
     static get pluginName(): "TableCellPropertiesEditing";
+    /**
+     * @inheritDoc
+     * @internal
+     */
+    static get licenseFeatureCode(): string;
     /**
      * @inheritDoc
      */
     static get isOfficialPlugin(): true;
+    /**
+     * @inheritDoc
+     */
+    static get isPremiumPlugin(): true;
     /**
      * @inheritDoc
      */

package/src/tablecellproperties/tablecellpropertiesediting.js

@@ -48,12 +48,25 @@ export class TableCellPropertiesEditing extends Plugin {
     static get pluginName() {
         return 'TableCellPropertiesEditing';
     }
+    /**
+     * @inheritDoc
+     * @internal
+     */
+    static get licenseFeatureCode() {
+        return 'TCP';
+    }
     /**
      * @inheritDoc
      */
     static get isOfficialPlugin() {
         return true;
     }
+    /**
+     * @inheritDoc
+     */
+    static get isPremiumPlugin() {
+        return true;
+    }
     /**
      * @inheritDoc
      */

package/src/tablecolumnresize/tablecolumnresizeediting.d.ts

@@ -47,10 +47,19 @@ export declare class TableColumnResizeEditing extends Plugin {
      * @inheritDoc
      */
     static get pluginName(): "TableColumnResizeEditing";
+    /**
+     * @inheritDoc
+     * @internal
+     */
+    static get licenseFeatureCode(): string;
     /**
      * @inheritDoc
      */
     static get isOfficialPlugin(): true;
+    /**
+     * @inheritDoc
+     */
+    static get isPremiumPlugin(): true;
     /**
      * @inheritDoc
      */

package/src/tablecolumnresize/tablecolumnresizeediting.js

@@ -54,12 +54,25 @@ export class TableColumnResizeEditing extends Plugin {
     static get pluginName() {
         return 'TableColumnResizeEditing';
     }
+    /**
+     * @inheritDoc
+     * @internal
+     */
+    static get licenseFeatureCode() {
+        return 'TCR';
+    }
     /**
      * @inheritDoc
      */
     static get isOfficialPlugin() {
         return true;
     }
+    /**
+     * @inheritDoc
+     */
+    static get isPremiumPlugin() {
+        return true;
+    }
     /**
      * @inheritDoc
      */

package/src/tablekeyboard.js

@@ -79,12 +79,9 @@ export class TableKeyboard extends Plugin {
         if (!selectedElement || !selectedElement.is('element', 'table')) {
             return;
         }
-        domEventData.preventDefault();
+        // For backward compatibility stop propagation of this DOM event.
+        // The default tab handling from widgets handles navigation.
         domEventData.stopPropagation();
-        bubblingEventInfo.stop();
-        editor.model.change(writer => {
-            writer.setSelection(writer.createRangeIn(selectedElement.getChild(0).getChild(0)));
-        });
     }
     /**
      * Handles {@link module:engine/view/document~ViewDocument#event:tab tab} events for the <kbd>Tab</kbd> key executed
@@ -103,52 +100,18 @@ export class TableKeyboard extends Plugin {
         if (!tableCell) {
             return;
         }
-        domEventData.preventDefault();
+        // For backward compatibility stop propagation of this DOM event.
+        // The default tab handling from widgets handles navigation.
         domEventData.stopPropagation();
-        bubblingEventInfo.stop();
         const tableRow = tableCell.parent;
         const table = tableRow.parent;
         const currentRowIndex = table.getChildIndex(tableRow);
         const currentCellIndex = tableRow.getChildIndex(tableCell);
-        const isFirstCellInRow = currentCellIndex === 0;
-        if (!isForward && isFirstCellInRow && currentRowIndex === 0) {
-            // Set the selection over the whole table if the selection was in the first table cell.
-            editor.model.change(writer => {
-                writer.setSelection(writer.createRangeOn(table));
-            });
-            return;
-        }
         const isLastCellInRow = currentCellIndex === tableRow.childCount - 1;
         const isLastRow = currentRowIndex === tableUtils.getRows(table) - 1;
         if (isForward && isLastRow && isLastCellInRow) {
             editor.execute('insertTableRowBelow');
-            // Check if the command actually added a row. If `insertTableRowBelow` execution didn't add a row (because it was disabled
-            // or it got overwritten) set the selection over the whole table to mirror the first cell case.
-            if (currentRowIndex === tableUtils.getRows(table) - 1) {
-                editor.model.change(writer => {
-                    writer.setSelection(writer.createRangeOn(table));
-                });
-                return;
-            }
         }
-        let cellToFocus;
-        // Move to the first cell in the next row.
-        if (isForward && isLastCellInRow) {
-            const nextRow = table.getChild(currentRowIndex + 1);
-            cellToFocus = nextRow.getChild(0);
-        }
-        // Move to the last cell in the previous row.
-        else if (!isForward && isFirstCellInRow) {
-            const previousRow = table.getChild(currentRowIndex - 1);
-            cellToFocus = previousRow.getChild(previousRow.childCount - 1);
-        }
-        // Move to the next/previous cell.
-        else {
-            cellToFocus = tableRow.getChild(currentCellIndex + (isForward ? 1 : -1));
-        }
-        editor.model.change(writer => {
-            writer.setSelection(writer.createRangeIn(cellToFocus));
-        });
     }
     /**
      * Handles {@link module:engine/view/document~ViewDocument#event:keydown keydown} events.

package/src/tablelayout/tablelayoutediting.d.ts

@@ -16,6 +16,11 @@ export declare class TableLayoutEditing extends Plugin {
      * @inheritDoc
      */
     static get pluginName(): "TableLayoutEditing";
+    /**
+     * @inheritDoc
+     * @internal
+     */
+    static get licenseFeatureCode(): string;
     /**
      * @inheritDoc
      */
@@ -24,6 +29,10 @@ export declare class TableLayoutEditing extends Plugin {
      * @inheritDoc
      */
     static get isOfficialPlugin(): true;
+    /**
+     * @inheritDoc
+     */
+    static get isPremiumPlugin(): true;
     /**
      * @inheritDoc
      */

package/src/tablelayout/tablelayoutediting.js

@@ -22,6 +22,13 @@ export class TableLayoutEditing extends Plugin {
     static get pluginName() {
         return 'TableLayoutEditing';
     }
+    /**
+     * @inheritDoc
+     * @internal
+     */
+    static get licenseFeatureCode() {
+        return 'TL';
+    }
     /**
      * @inheritDoc
      */
@@ -34,6 +41,12 @@ export class TableLayoutEditing extends Plugin {
     static get isOfficialPlugin() {
         return true;
     }
+    /**
+     * @inheritDoc
+     */
+    static get isPremiumPlugin() {
+        return true;
+    }
     /**
      * @inheritDoc
      */

package/src/tableproperties/tablepropertiesediting.d.ts

@@ -29,10 +29,19 @@ export declare class TablePropertiesEditing extends Plugin {
      * @inheritDoc
      */
     static get pluginName(): "TablePropertiesEditing";
+    /**
+     * @inheritDoc
+     * @internal
+     */
+    static get licenseFeatureCode(): string;
     /**
      * @inheritDoc
      */
     static get isOfficialPlugin(): true;
+    /**
+     * @inheritDoc
+     */
+    static get isPremiumPlugin(): true;
     /**
      * @inheritDoc
      */

package/src/tableproperties/tablepropertiesediting.js

@@ -43,12 +43,25 @@ export class TablePropertiesEditing extends Plugin {
     static get pluginName() {
         return 'TablePropertiesEditing';
     }
+    /**
+     * @inheritDoc
+     * @internal
+     */
+    static get licenseFeatureCode() {
+        return 'TCP';
+    }
     /**
      * @inheritDoc
      */
     static get isOfficialPlugin() {
         return true;
     }
+    /**
+     * @inheritDoc
+     */
+    static get isPremiumPlugin() {
+        return true;
+    }
     /**
      * @inheritDoc
      */

package/src/utils/ui/widget.d.ts

@@ -5,7 +5,7 @@
 /**
  * @module table/utils/ui/widget
  */
-import type { ViewDocumentSelection, ViewElement } from 'ckeditor5/src/engine.js';
+import type { ViewDocumentSelection, ViewElement, ViewNode } from 'ckeditor5/src/engine.js';
 /**
  * Depending on the position of the selection either return the selected table or the table higher in the hierarchy.
  *
@@ -24,3 +24,9 @@ export declare function getSelectedTableWidget(selection: ViewDocumentSelection)
  * @internal
  */
 export declare function getTableWidgetAncestor(selection: ViewDocumentSelection): ViewElement | null;
+/**
+ * Checks if a given view element is a table widget.
+ *
+ * @internal
+ */
+export declare function isTableWidget(viewNode: ViewNode): boolean;

package/src/utils/ui/widget.js

@@ -48,7 +48,9 @@ export function getTableWidgetAncestor(selection) {
 }
 /**
  * Checks if a given view element is a table widget.
+ *
+ * @internal
  */
-function isTableWidget(viewElement) {
-    return !!viewElement.getCustomProperty('table') && isWidget(viewElement);
+export function isTableWidget(viewNode) {
+    return viewNode.is('element') && !!viewNode.getCustomProperty('table') && isWidget(viewNode);
 }

package/theme/table.css

@@ -14,24 +14,8 @@
 		text-align: left;
 	}
 
-	& figure.table:not(.layout-table) {
-		display: table;
-
-		& > table {
-			width: 100%;
-			height: 100%;
-		}
-	}
-
-	& .table:not(.layout-table) {
-		/* Give the table widget some air and center it horizontally */
-		/* The first value should be equal to --ck-spacing-large variable if used in the editor context
-		to avoid the content jumping (See https://github.com/ckeditor/ckeditor5/issues/9825). */
-		margin: 0.9em auto;
-	}
-
-	& table.table:not(.layout-table),
-	& figure.table:not(.layout-table) > table {
+	& :where(table).table:where(:not(.layout-table)),
+	& :where(figure).table:where(:not(.layout-table)) > table {
 		/* The table cells should have slight borders */
 		border-collapse: collapse;
 		border-spacing: 0;
@@ -40,13 +24,12 @@
 		Also see https://github.com/ckeditor/ckeditor5-table/issues/50. */
 		border: 1px double hsl(0, 0%, 70%);
 
-		& > thead,
-		& > tbody {
+		& > :where(thead, tbody) {
 			/* The linter is disabled here because linter is confused when resolving the `table.table:not(.layout-table)`
 			and `figure.table:not(.layout-table) > table` selectors combined with below selectors.
 			There is no need to split it into two large structures with same code just to make linter happy. */
 			/* stylelint-disable no-descending-specificity */
-			& > tr {
+			& > :where(tr) {
 				& > th {
 					font-weight: bold;
 					background: hsla(0, 0%, 0%, 5%);
@@ -86,6 +69,22 @@
 			}
 		}
 	}
+
+	& figure.table:not(.layout-table) {
+		display: table;
+
+		& > table {
+			width: 100%;
+			height: 100%;
+		}
+	}
+
+	& .table:not(.layout-table) {
+		/* Give the table widget some air and center it horizontally */
+		/* The first value should be equal to --ck-spacing-large variable if used in the editor context
+		to avoid the content jumping (See https://github.com/ckeditor/ckeditor5/issues/9825). */
+		margin: 0.9em auto;
+	}
 }
 
 @media print {