vaadin · web-padawan · Nov 3, 2021 · Oct 15, 2021
diff --git a/packages/board/src/vaadin-board.d.ts b/packages/board/src/vaadin-board.d.ts
@@ -25,8 +25,6 @@ import { ElementMixin } from '@vaadin/component-base/src/element-mixin.js';
  * ```
  */
 declare class Board extends ElementMixin(HTMLElement) {
-  static _finalizeClass(): void;
-
   /**
    * Redraws the board and all rows inside it, if necessary.
    *

diff --git a/packages/charts/src/vaadin-chart-series.d.ts b/packages/charts/src/vaadin-chart-series.d.ts
@@ -160,8 +160,6 @@ declare class ChartSeries extends HTMLElement {
    */
   additionalOptions: SeriesOptionsType | null | undefined;
 
-  _series: Series | undefined;
-
   /**
    * Method to attach a series object of type `Highcharts.Series`.
    *

diff --git a/packages/charts/src/vaadin-chart.d.ts b/packages/charts/src/vaadin-chart.d.ts
@@ -541,29 +541,6 @@ declare class Chart extends ThemableMixin(ElementMixin(HTMLElement)) {
    */
   polar: boolean | null | undefined;
 
-  /**
-   * Search for axis with given `id`.
-   *
-   * @param id contains the id that will be searched
-   * @param isXAxis indicates if it will remove x or y axes. Defaults to `false`.
-   */
-  __getAxis(id: string, isXAxis: boolean): Axis | null;
-
-  /**
-   * Add an axis with given options
-   *
-   * @param options axis options
-   * @param isXAxis indicates if axis is X (`true`) or Y (`false`). Defaults to `false`.
-   */
-  __addAxis(options: object | null, isXAxis: boolean): Axis;
-
-  /**
-   * Iterates over axes (y or x) and removes whenever it doesn't contain any series and was created for unit
-   *
-   * @param isXAxis indicates if it will remove x or y axes. Defaults to `false`.
-   */
-  __removeAxisIfEmpty(isXAxis: boolean): void;
-
   /**
    * Update the chart configuration.
    * This JSON API provides a simple single-argument alternative to the configuration property.

diff --git a/packages/component-base/src/focus-mixin.js b/packages/component-base/src/focus-mixin.js
@@ -92,7 +92,7 @@ export const FocusMixin = dedupingMixin(
       /**
        * Override to define if the field receives focus based on the event.
        *
-       * @param {FocusEvent} event
+       * @param {FocusEvent} _event
        * @return {boolean}
        * @protected
        */
@@ -103,7 +103,7 @@ export const FocusMixin = dedupingMixin(
       /**
        * Override to define if the field loses focus based on the event.
        *
-       * @param {FocusEvent} event
+       * @param {FocusEvent} _event
        * @return {boolean}
        * @protected
        */

diff --git a/packages/cookie-consent/src/vaadin-cookie-consent.d.ts b/packages/cookie-consent/src/vaadin-cookie-consent.d.ts
@@ -70,11 +70,6 @@ declare class CookieConsent extends ElementMixin(HTMLElement) {
    * @attr {string} cookie-name
    */
   cookieName: string;
-
-  /**
-   * Shows the popup even if the user has seen it before.
-   */
-  _show(): void;
 }
 
 declare global {

diff --git a/packages/crud/src/vaadin-crud-form.d.ts b/packages/crud/src/vaadin-crud-form.d.ts
@@ -14,13 +14,6 @@ declare class CrudForm<Item> extends IncludedMixin(FormLayout) {
    * The item being edited.
    */
   item: Item | null | undefined;
-
-  /**
-   * Auto-generate form fields based on the JSON structure of the object provided.
-   *
-   * If not called, the method will be executed the first time an item is assigned.
-   */
-  _configure(object: Item): void;
 }
 
 declare global {

diff --git a/packages/grid-pro/src/vaadin-grid-pro-edit-column.d.ts b/packages/grid-pro/src/vaadin-grid-pro-edit-column.d.ts
@@ -70,15 +70,13 @@ declare class GridProEditColumn<TItem = GridDefaultItem> extends GridColumn<TIte
    */
   editorValuePath: string;
 
-  _getEditorTagName(cell: HTMLElement): string;
+  protected _getEditorComponent(cell: HTMLElement): HTMLElement | null;
 
-  _getEditorComponent(cell: HTMLElement): HTMLElement | null;
+  protected _getEditorValue(editor: HTMLElement): unknown | null;
 
-  _getEditorValue(editor: HTMLElement): unknown | null;
+  protected _startCellEdit(cell: HTMLElement, model: GridItemModel<TItem>): void;
 
-  _startCellEdit(cell: HTMLElement, model: GridItemModel<TItem>): void;
-
-  _stopCellEdit(cell: HTMLElement, model: GridItemModel<TItem>): void;
+  protected _stopCellEdit(cell: HTMLElement, model: GridItemModel<TItem>): void;
 }
 
 declare global {

diff --git a/packages/grid-pro/src/vaadin-grid-pro.d.ts b/packages/grid-pro/src/vaadin-grid-pro.d.ts
@@ -75,8 +75,6 @@ export interface GridProEventMap<TItem>
  * @fires {CustomEvent} selected-items-changed - Fired when the `selectedItems` property changes.
  */
 declare class GridPro<TItem = GridDefaultItem> extends Grid<TItem> {
-  static _finalizeClass(): void;
-
   addEventListener<K extends keyof GridProEventMap<TItem>>(
     type: K,
     listener: (this: GridPro<TItem>, ev: GridProEventMap<TItem>[K]) => void,

diff --git a/packages/list-box/src/vaadin-list-box.d.ts b/packages/list-box/src/vaadin-list-box.d.ts
@@ -61,8 +61,6 @@ export interface ListBoxEventMap extends HTMLElementEventMap, ListBoxCustomEvent
 declare class ListBox extends MultiSelectListMixin(ThemableMixin(ElementMixin(HTMLElement))) {
   focused: Element | null;
 
-  readonly _scrollerElement: HTMLElement;
-
   addEventListener<K extends keyof ListBoxEventMap>(
     type: K,
     listener: (this: ListBox, ev: ListBoxEventMap[K]) => void,

diff --git a/packages/tabs/src/vaadin-tab.d.ts b/packages/tabs/src/vaadin-tab.d.ts
@@ -29,9 +29,7 @@ import { ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mix
  *
  * See [Styling Components](https://vaadin.com/docs/latest/ds/customization/styling-components) documentation.
  */
-declare class Tab extends ElementMixin(ThemableMixin(ItemMixin(HTMLElement))) {
-  _onKeyup(event: KeyboardEvent): void;
-}
+declare class Tab extends ElementMixin(ThemableMixin(ItemMixin(HTMLElement))) {}
 
 declare global {
   interface HTMLElementTagNameMap {

diff --git a/packages/tabs/src/vaadin-tabs.d.ts b/packages/tabs/src/vaadin-tabs.d.ts
@@ -62,8 +62,6 @@ export interface TabsEventMap extends HTMLElementEventMap, TabsCustomEventMap {}
  * @fires {CustomEvent} selected-changed - Fired when the `selected` property changes.
  */
 declare class Tabs extends ElementMixin(ListMixin(ThemableMixin(HTMLElement))) {
-  readonly _scrollerElement: HTMLElement;
-
   /**
    * The index of the selected tab.
    */
@@ -74,8 +72,6 @@ declare class Tabs extends ElementMixin(ListMixin(ThemableMixin(HTMLElement))) {
    */
   orientation: TabsOrientation;
 
-  readonly _scrollOffset: number;
-
   addEventListener<K extends keyof TabsEventMap>(
     type: K,
     listener: (this: Tabs, ev: TabsEventMap[K]) => void,

diff --git a/packages/upload/src/vaadin-upload.d.ts b/packages/upload/src/vaadin-upload.d.ts
@@ -398,20 +398,6 @@ declare class Upload extends ThemableMixin(ElementMixin(HTMLElement)) {
    */
   uploadFiles(files?: UploadFile | UploadFile[]): void;
 
-  /**
-   * Add the file for uploading. Called internally for each file after picking files from dialog or dropping files.
-   *
-   * @param file File being added
-   */
-  _addFile(file: UploadFile): void;
-
-  /**
-   * Remove file from upload list. Called internally if file upload was canceled.
-   *
-   * @param file File to remove
-   */
-  _removeFile(file: UploadFile): void;
-
   addEventListener<K extends keyof UploadEventMap>(
     type: K,
     listener: (this: Upload, ev: UploadEventMap[K]) => void,

diff --git a/packages/vaadin-overlay/src/vaadin-overlay.d.ts b/packages/vaadin-overlay/src/vaadin-overlay.d.ts
@@ -111,11 +111,6 @@ export type OverlayEventMap = HTMLElementEventMap & OverlayElementEventMap;
  * @fires {CustomEvent} opened-changed - Fired when the `opened` property changes.
  */
 declare class OverlayElement extends ThemableMixin(DirMixin(HTMLElement)) {
-  /**
-   * returns true if this is the last one in the opened overlays stack
-   */
-  readonly _last: boolean;
-
   /**
    * When true, the overlay is visible and attached to body.
    */
@@ -184,38 +179,8 @@ declare class OverlayElement extends ThemableMixin(DirMixin(HTMLElement)) {
    */
   restoreFocusOnClose: boolean;
 
-  _setTemplateFromNodes(nodes: Element[]): void;
-
   close(sourceEvent?: Event | null): void;
 
-  _ensureTemplatized(): void;
-
-  _shouldAnimate(): boolean;
-
-  _enqueueAnimation(type: string, callback: Function | null): void;
-
-  _flushAnimation(type: string): void;
-
-  _animatedOpening(): void;
-
-  _attachOverlay(): void;
-
-  _animatedClosing(): void;
-
-  _detachOverlay(): void;
-
-  _addGlobalListeners(): void;
-
-  _enterModalState(): void;
-
-  _removeGlobalListeners(): void;
-
-  _exitModalState(): void;
-
-  _removeOldContent(): void;
-
-  _stampOverlayTemplate(template: HTMLTemplateElement, instanceProps: object | null): void;
-
   /**
    * Requests an update for the content of the overlay.
    * While performing the update, it invokes the renderer passed in the `renderer` property.
@@ -224,18 +189,6 @@ declare class OverlayElement extends ThemableMixin(DirMixin(HTMLElement)) {
    */
   requestContentUpdate(): void;
 
-  _isFocused(element: Element | null): boolean;
-
-  _focusedIndex(elements: Array<Element | null> | null): number;
-
-  _cycleTab(increment: number, index: number | undefined): void;
-
-  _getFocusableElements(): HTMLElement[];
-
-  _getActiveElement(): Element;
-
-  _deepContains(node: Node): boolean;
-
   /**
    * Brings the overlay as visually the frontmost one
    */