@vaadin/vaadin-overlay 22.0.0-alpha9 → 23.0.0-alpha1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/vaadin-overlay",
-  "version": "22.0.0-alpha9",
+  "version": "23.0.0-alpha1",
   "publishConfig": {
     "access": "public"
   },
@@ -34,20 +34,20 @@
   ],
   "dependencies": {
     "@polymer/polymer": "^3.0.0",
-    "@vaadin/component-base": "22.0.0-alpha9",
-    "@vaadin/vaadin-lumo-styles": "22.0.0-alpha9",
-    "@vaadin/vaadin-material-styles": "22.0.0-alpha9",
-    "@vaadin/vaadin-themable-mixin": "22.0.0-alpha9"
+    "@vaadin/component-base": "23.0.0-alpha1",
+    "@vaadin/vaadin-lumo-styles": "23.0.0-alpha1",
+    "@vaadin/vaadin-material-styles": "23.0.0-alpha1",
+    "@vaadin/vaadin-themable-mixin": "23.0.0-alpha1"
   },
   "devDependencies": {
     "@esm-bundle/chai": "^4.3.4",
     "@polymer/iron-overlay-behavior": "^3.0.0",
-    "@vaadin/button": "22.0.0-alpha9",
-    "@vaadin/radio-group": "22.0.0-alpha9",
-    "@vaadin/testing-helpers": "^0.3.0",
-    "@vaadin/text-field": "22.0.0-alpha9",
+    "@vaadin/button": "23.0.0-alpha1",
+    "@vaadin/radio-group": "23.0.0-alpha1",
+    "@vaadin/testing-helpers": "^0.3.2",
+    "@vaadin/text-field": "23.0.0-alpha1",
     "lit": "^2.0.0",
     "sinon": "^9.2.1"
   },
-  "gitHead": "6e8c899dc65918f97e3c0acb2076122c4b2ef274"
+  "gitHead": "fbcb07328fdf88260e3b461088d207426b21c710"
 }

package/src/vaadin-overlay.d.ts

@@ -61,8 +61,7 @@ export type OverlayEventMap = HTMLElementEventMap & OverlayElementEventMap;
  *
  * The overlay provides `forwardHostProp` when calling
  * `Polymer.Templatize.templatize` for the template, so that the bindings
- * from the parent scope propagate to the content.  You can also pass
- * custom `instanceProps` object using the `instanceProps` property.
+ * from the parent scope propagate to the content.
  *
  * ```html
  * <vaadin-overlay>
@@ -70,10 +69,6 @@ export type OverlayEventMap = HTMLElementEventMap & OverlayElementEventMap;
  * </vaadin-overlay>
  * ```
  *
- * **NOTE:** when using `instanceProps`: because of the Polymer limitation,
- * every template can only be templatized once, so it is important
- * to set `instanceProps` before the `template` is assigned to the overlay.
- *
  * ### Styling
  *
  * To style the overlay content, use styles in the parent scope:
@@ -111,11 +106,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.
    */
@@ -141,11 +131,6 @@ declare class OverlayElement extends ThemableMixin(DirMixin(HTMLElement)) {
    */
   template: HTMLTemplateElement | null | undefined;
 
-  /**
-   * Optional argument for `Polymer.Templatize.templatize`.
-   */
-  instanceProps: object | null | undefined;
-
   /**
    * References the content container after the template is stamped.
    */
@@ -184,38 +169,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 +179,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
    */

package/src/vaadin-overlay.js

@@ -3,10 +3,11 @@
  * Copyright (c) 2021 Vaadin Ltd.
  * This program is available under Apache License Version 2.0, available at https://vaadin.com/license/
  */
-import { PolymerElement, html } from '@polymer/polymer/polymer-element.js';
-import { templatize } from '@polymer/polymer/lib/utils/templatize.js';
-import { afterNextRender } from '@polymer/polymer/lib/utils/render-status.js';
 import { FlattenedNodesObserver } from '@polymer/polymer/lib/utils/flattened-nodes-observer.js';
+import { afterNextRender } from '@polymer/polymer/lib/utils/render-status.js';
+import { templatize } from '@polymer/polymer/lib/utils/templatize.js';
+import { html, PolymerElement } from '@polymer/polymer/polymer-element.js';
+import { isIOS } from '@vaadin/component-base/src/browser-utils.js';
 import { DirMixin } from '@vaadin/component-base/src/dir-mixin.js';
 import { ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mixin.js';
 import { FocusablesHelper } from './vaadin-focusables-helper.js';
@@ -54,18 +55,7 @@ import { FocusablesHelper } from './vaadin-focusables-helper.js';
  *
  * The overlay provides `forwardHostProp` when calling
  * `Polymer.Templatize.templatize` for the template, so that the bindings
- * from the parent scope propagate to the content.  You can also pass
- * custom `instanceProps` object using the `instanceProps` property.
- *
- * ```html
- * <vaadin-overlay>
- *   <template>Overlay content</template>
- * </vaadin-overlay>
- * ```
- *
- * **NOTE:** when using `instanceProps`: because of the Polymer limitation,
- * every template can only be templatized once, so it is important
- * to set `instanceProps` before the `template` is assigned to the overlay.
+ * from the parent scope propagate to the content.
  *
  * ### Styling
  *
@@ -226,13 +216,6 @@ class OverlayElement extends ThemableMixin(DirMixin(PolymerElement)) {
         notify: true
       },
 
-      /**
-       * Optional argument for `Polymer.Templatize.templatize`.
-       */
-      instanceProps: {
-        type: Object
-      },
-
       /**
        * References the content container after the template is stamped.
        * @type {!HTMLElement | undefined}
@@ -329,9 +312,6 @@ class OverlayElement extends ThemableMixin(DirMixin(PolymerElement)) {
       /** @private */
       _oldTemplate: Object,
 
-      /** @private */
-      _oldInstanceProps: Object,
-
       /** @private */
       _oldRenderer: Object,
 
@@ -341,7 +321,7 @@ class OverlayElement extends ThemableMixin(DirMixin(PolymerElement)) {
   }
 
   static get observers() {
-    return ['_templateOrRendererChanged(template, renderer, owner, model, instanceProps, opened)'];
+    return ['_templateOrRendererChanged(template, renderer, owner, model, opened)'];
   }
 
   constructor() {
@@ -359,7 +339,7 @@ class OverlayElement extends ThemableMixin(DirMixin(PolymerElement)) {
     this._boundIronOverlayCanceledListener = this._ironOverlayCanceled.bind(this);
 
     /* c8 ignore next 3 */
-    if (/iPad|iPhone|iPod/.test(navigator.userAgent)) {
+    if (isIOS) {
       this._boundIosResizeListener = () => this._detectIosNavbar();
     }
   }
@@ -824,15 +804,13 @@ class OverlayElement extends ThemableMixin(DirMixin(PolymerElement)) {
 
   /**
    * @param {!HTMLTemplateElement} template
-   * @param {object} instanceProps
    * @protected
    */
-  _stampOverlayTemplate(template, instanceProps) {
+  _stampOverlayTemplate(template) {
     this._removeOldContent();
 
     if (!template._Templatizer) {
       template._Templatizer = templatize(template, this, {
-        instanceProps: instanceProps,
         forwardHostProp: function (prop, value) {
           if (this._instance) {
             this._instance.forwardHostProp(prop, value);
@@ -885,7 +863,7 @@ class OverlayElement extends ThemableMixin(DirMixin(PolymerElement)) {
   }
 
   /** @private */
-  _templateOrRendererChanged(template, renderer, owner, model, instanceProps, opened) {
+  _templateOrRendererChanged(template, renderer, owner, model, opened) {
     if (template && renderer) {
       this._removeNewRendererOrTemplate(template, this._oldTemplate, renderer, this._oldRenderer);
       throw new Error('You should only use either a renderer or a template for overlay content');
@@ -895,8 +873,7 @@ class OverlayElement extends ThemableMixin(DirMixin(PolymerElement)) {
     this._oldModel = model;
     this._oldOwner = owner;
 
-    const templateOrInstancePropsChanged = this._oldInstanceProps !== instanceProps || this._oldTemplate !== template;
-    this._oldInstanceProps = instanceProps;
+    const templateChanged = this._oldTemplate !== template;
     this._oldTemplate = template;
 
     const rendererChanged = this._oldRenderer !== renderer;
@@ -914,8 +891,8 @@ class OverlayElement extends ThemableMixin(DirMixin(PolymerElement)) {
       delete this.content._$litPart$;
     }
 
-    if (template && templateOrInstancePropsChanged) {
-      this._stampOverlayTemplate(template, instanceProps);
+    if (template && templateChanged) {
+      this._stampOverlayTemplate(template);
     } else if (renderer && (rendererChanged || openedChanged || ownerOrModelChanged)) {
       if (opened) {
         this.requestContentUpdate();

package/theme/lumo/vaadin-overlay-styles.js

@@ -1,4 +1,4 @@
-import { registerStyles } from '@vaadin/vaadin-themable-mixin/register-styles.js';
 import { overlay } from '@vaadin/vaadin-lumo-styles/mixins/overlay.js';
+import { registerStyles } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mixin.js';
 
 registerStyles('vaadin-overlay', overlay, { moduleId: 'lumo-vaadin-overlay' });

package/theme/material/vaadin-overlay-styles.js

@@ -1,4 +1,4 @@
-import { registerStyles } from '@vaadin/vaadin-themable-mixin/register-styles.js';
 import { overlay } from '@vaadin/vaadin-material-styles/mixins/overlay.js';
+import { registerStyles } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mixin.js';
 
 registerStyles('vaadin-overlay', overlay, { moduleId: 'material-vaadin-overlay' });