refactor!: mark A11yMixin methods as private

Author: vursen

packages/grid/src/vaadin-grid-a11y-mixin.js

@@ -22,67 +22,67 @@ export const A11yMixin = (superClass) =>
       };
     }
     static get observers() {
-      return ['_a11yUpdateGridSize(size, _columnTree)'];
+      return ['__updateA11yGridSize(size, _columnTree)'];
     }
 
     /** @private */
-    _a11yGetHeaderRowCount(_columnTree) {
+    __getA11yHeaderRowCount(_columnTree) {
       return _columnTree.filter((level) =>
         level.some((col) => col.headerRenderer || (col.path && col.header !== null) || col.header),
       ).length;
     }
 
     /** @private */
-    _a11yGetFooterRowCount(_columnTree) {
+    __getA11yFooterRowCount(_columnTree) {
       return _columnTree.filter((level) => level.some((col) => col.headerRenderer)).length;
     }
 
     /** @private */
-    _a11yUpdateGridSize(size, _columnTree) {
-      if (size === undefined || _columnTree === undefined) {
+    __updateA11yGridSize(size, columnTree) {
+      if (size === undefined || columnTree === undefined) {
         return;
       }
 
-      const bodyColumns = _columnTree[_columnTree.length - 1];
+      const bodyColumns = columnTree[columnTree.length - 1];
       this.$.table.setAttribute(
         'aria-rowcount',
-        size + this._a11yGetHeaderRowCount(_columnTree) + this._a11yGetFooterRowCount(_columnTree),
+        size + this.__getA11yHeaderRowCount(columnTree) + this.__getA11yFooterRowCount(columnTree),
       );
       this.$.table.setAttribute('aria-colcount', (bodyColumns && bodyColumns.length) || 0);
 
-      this._a11yUpdateHeaderRows();
-      this._a11yUpdateFooterRows();
+      this.__updateA11yHeaderRows();
+      this.__updateA11yFooterRows();
     }
 
-    /** @protected */
-    _a11yUpdateHeaderRows() {
+    /** @private */
+    __updateA11yHeaderRows() {
       iterateChildren(this.$.header, (headerRow, index) => {
         headerRow.setAttribute('aria-rowindex', index + 1);
       });
     }
 
-    /** @protected */
-    _a11yUpdateFooterRows() {
+    /** @private */
+    __updateA11yFooterRows() {
       iterateChildren(this.$.footer, (footerRow, index) => {
-        footerRow.setAttribute('aria-rowindex', this._a11yGetHeaderRowCount(this._columnTree) + this.size + index + 1);
+        footerRow.setAttribute('aria-rowindex', this.__getA11yHeaderRowCount(this._columnTree) + this.size + index + 1);
       });
     }
 
     /**
      * @param {!HTMLElement} row
      * @param {number} index
-     * @protected
+     * @private
      */
-    _a11yUpdateRowRowindex(row, index) {
-      row.setAttribute('aria-rowindex', index + this._a11yGetHeaderRowCount(this._columnTree) + 1);
+    __updateA11yRowRowindex(row, index) {
+      row.setAttribute('aria-rowindex', index + this.__getA11yHeaderRowCount(this._columnTree) + 1);
     }
 
     /**
      * @param {!HTMLElement} row
      * @param {boolean} selected
-     * @protected
+     * @private
      */
-    _a11yUpdateRowSelected(row, selected) {
+    __updateA11yRowSelected(row, selected) {
       // Jaws reads selection only for rows, NVDA only for cells
       row.setAttribute('aria-selected', Boolean(selected));
       iterateRowCells(row, (cell) => {
@@ -92,9 +92,9 @@ export const A11yMixin = (superClass) =>
 
     /**
      * @param {!HTMLElement} row
-     * @protected
+     * @private
      */
-    _a11yUpdateRowExpanded(row) {
+    __updateA11yRowExpanded(row) {
       if (this.__isRowExpandable(row)) {
         row.setAttribute('aria-expanded', 'false');
       } else if (this.__isRowCollapsible(row)) {
@@ -107,9 +107,9 @@ export const A11yMixin = (superClass) =>
     /**
      * @param {!HTMLElement} row
      * @param {number} level
-     * @protected
+     * @private
      */
-    _a11yUpdateRowLevel(row, level) {
+    __updateA11yRowLevel(row, level) {
       // Set level for the expandable rows itself, and all the nested rows.
       if (level > 0 || this.__isRowCollapsible(row) || this.__isRowExpandable(row)) {
         row.setAttribute('aria-level', level + 1);
@@ -121,9 +121,9 @@ export const A11yMixin = (superClass) =>
     /**
      * @param {!HTMLElement} row
      * @param {!HTMLElement} detailsCell
-     * @protected
+     * @private
      */
-    _a11ySetRowDetailsCell(row, detailsCell) {
+    __setA11yRowDetailsCell(row, detailsCell) {
       iterateRowCells(row, (cell) => {
         if (cell !== detailsCell) {
           cell.setAttribute('aria-controls', detailsCell.id);
@@ -134,14 +134,14 @@ export const A11yMixin = (superClass) =>
     /**
      * @param {!HTMLElement} row
      * @param {number} colspan
-     * @protected
+     * @private
      */
-    _a11yUpdateCellColspan(cell, colspan) {
+    __updateA11yCellColspan(cell, colspan) {
       cell.setAttribute('aria-colspan', Number(colspan));
     }
 
-    /** @protected */
-    _a11yUpdateSorters() {
+    /** @private */
+    __updateA11ySorters() {
       Array.from(this.querySelectorAll('vaadin-grid-sorter')).forEach((sorter) => {
         let cellContent = sorter.parentNode;
         while (cellContent && cellContent.localName !== 'vaadin-grid-cell-content') {

packages/grid/src/vaadin-grid-column-group-mixin.js

@@ -314,13 +314,13 @@ export const GridColumnGroupMixin = (superClass) =>
       if (headerCell) {
         headerCell.setAttribute('colspan', colSpan);
         if (this._grid) {
-          this._grid._a11yUpdateCellColspan(headerCell, colSpan);
+          this._grid.__updateA11yCellColspan(headerCell, colSpan);
         }
       }
       if (footerCell) {
         footerCell.setAttribute('colspan', colSpan);
         if (this._grid) {
-          this._grid._a11yUpdateCellColspan(footerCell, colSpan);
+          this._grid.__updateA11yCellColspan(footerCell, colSpan);
         }
       }
     }

packages/grid/src/vaadin-grid-mixin.js

@@ -521,7 +521,7 @@ export const GridMixin = (superClass) =>
               row.appendChild(detailsCell);
               // Cache the details cell reference
               row.__detailsCell = detailsCell;
-              this._a11ySetRowDetailsCell(row, detailsCell);
+              this.__setA11yRowDetailsCell(row, detailsCell);
               detailsCell._vacant = false;
             }
 
@@ -644,7 +644,7 @@ export const GridMixin = (superClass) =>
 
       this._updateRowOrderParts(row, index);
 
-      this._a11yUpdateRowRowindex(row, index);
+      this.__updateA11yRowRowindex(row, index);
 
       this.__ensureRowItem(row);
       this.__ensureRowHierarchy(row);
@@ -738,8 +738,8 @@ export const GridMixin = (superClass) =>
       this._frozenCellsChanged();
       this._updateFirstAndLastColumn();
       this._resetKeyboardNavigation();
-      this._a11yUpdateHeaderRows();
-      this._a11yUpdateFooterRows();
+      this.__updateA11yHeaderRows();
+      this.__updateA11yFooterRows();
       this.generateCellClassNames();
       this.generateCellPartNames();
       this.__updateHeaderAndFooter();
@@ -784,8 +784,8 @@ export const GridMixin = (superClass) =>
 
       this._toggleDetailsCell(row, model.detailsOpened);
 
-      this._a11yUpdateRowLevel(row, model.level);
-      this._a11yUpdateRowSelected(row, model.selected);
+      this.__updateA11yRowLevel(row, model.level);
+      this.__updateA11yRowSelected(row, model.selected);
 
       this._updateRowStateParts(row, model);
 
@@ -806,7 +806,7 @@ export const GridMixin = (superClass) =>
 
       this._updateDetailsCellHeight(row);
 
-      this._a11yUpdateRowExpanded(row, model.expanded);
+      this.__updateA11yRowExpanded(row, model.expanded);
     }
 
     /** @private */

packages/grid/src/vaadin-grid-sort-mixin.js

@@ -162,7 +162,7 @@ export const SortMixin = (superClass) =>
         this.__debounceClearCache();
       }
 
-      this._a11yUpdateSorters();
+      this.__updateA11ySorters();
 
       this._previousSorters = this._mapSorters();
     }