@vaadin/field-highlighter 24.7.0-alpha6 → 24.7.0-alpha8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/field-highlighter",
-  "version": "24.7.0-alpha6",
+  "version": "24.7.0-alpha8",
   "publishConfig": {
     "access": "public"
   },
@@ -34,30 +34,31 @@
   ],
   "dependencies": {
     "@polymer/polymer": "^3.0.0",
-    "@vaadin/a11y-base": "24.7.0-alpha6",
-    "@vaadin/component-base": "24.7.0-alpha6",
-    "@vaadin/overlay": "24.7.0-alpha6",
-    "@vaadin/vaadin-lumo-styles": "24.7.0-alpha6",
-    "@vaadin/vaadin-material-styles": "24.7.0-alpha6",
-    "@vaadin/vaadin-themable-mixin": "24.7.0-alpha6",
+    "@vaadin/a11y-base": "24.7.0-alpha8",
+    "@vaadin/component-base": "24.7.0-alpha8",
+    "@vaadin/overlay": "24.7.0-alpha8",
+    "@vaadin/vaadin-lumo-styles": "24.7.0-alpha8",
+    "@vaadin/vaadin-material-styles": "24.7.0-alpha8",
+    "@vaadin/vaadin-themable-mixin": "24.7.0-alpha8",
     "lit": "^3.0.0"
   },
   "devDependencies": {
-    "@vaadin/chai-plugins": "24.7.0-alpha6",
-    "@vaadin/checkbox": "24.7.0-alpha6",
-    "@vaadin/checkbox-group": "24.7.0-alpha6",
-    "@vaadin/combo-box": "24.7.0-alpha6",
-    "@vaadin/date-picker": "24.7.0-alpha6",
-    "@vaadin/date-time-picker": "24.7.0-alpha6",
-    "@vaadin/item": "24.7.0-alpha6",
-    "@vaadin/list-box": "24.7.0-alpha6",
-    "@vaadin/radio-group": "24.7.0-alpha6",
-    "@vaadin/select": "24.7.0-alpha6",
+    "@vaadin/chai-plugins": "24.7.0-alpha8",
+    "@vaadin/checkbox": "24.7.0-alpha8",
+    "@vaadin/checkbox-group": "24.7.0-alpha8",
+    "@vaadin/combo-box": "24.7.0-alpha8",
+    "@vaadin/date-picker": "24.7.0-alpha8",
+    "@vaadin/date-time-picker": "24.7.0-alpha8",
+    "@vaadin/item": "24.7.0-alpha8",
+    "@vaadin/list-box": "24.7.0-alpha8",
+    "@vaadin/radio-group": "24.7.0-alpha8",
+    "@vaadin/select": "24.7.0-alpha8",
+    "@vaadin/test-runner-commands": "24.7.0-alpha8",
     "@vaadin/testing-helpers": "^1.1.0",
-    "@vaadin/text-area": "24.7.0-alpha6",
-    "@vaadin/text-field": "24.7.0-alpha6",
-    "@vaadin/time-picker": "24.7.0-alpha6",
+    "@vaadin/text-area": "24.7.0-alpha8",
+    "@vaadin/text-field": "24.7.0-alpha8",
+    "@vaadin/time-picker": "24.7.0-alpha8",
     "sinon": "^18.0.0"
   },
-  "gitHead": "6255a512997a648da91fed37de4d5000809eaebf"
+  "gitHead": "d015035192480fcc8cc9df5d00a950f177b83c32"
 }

package/src/vaadin-field-highlighter-styles.js

@@ -65,7 +65,7 @@ export const userTagsOverlayStyles = css`
     box-shadow: none;
   }
 
-  [part='overlay'] {
+  :scope [part='overlay'] {
     box-shadow: none;
     background: transparent;
     position: relative;
@@ -86,7 +86,7 @@ export const userTagsOverlayStyles = css`
     right: -4px;
   }
 
-  [part='content'] {
+  :scope [part='content'] {
     padding: 0;
   }
 

package/src/vaadin-field-outline.js

@@ -3,9 +3,10 @@
  * Copyright (c) 2021 - 2025 Vaadin Ltd.
  * This program is available under Apache License Version 2.0, available at https://vaadin.com/license/
  */
-import { html, PolymerElement } from '@polymer/polymer/polymer-element.js';
+import { html, LitElement } from 'lit';
 import { defineCustomElement } from '@vaadin/component-base/src/define.js';
 import { DirMixin } from '@vaadin/component-base/src/dir-mixin.js';
+import { PolylitMixin } from '@vaadin/component-base/src/polylit-mixin.js';
 import { registerStyles, ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mixin.js';
 import { fieldOutlineStyles } from './vaadin-field-highlighter-styles.js';
 
@@ -19,15 +20,11 @@ registerStyles('vaadin-field-outline', fieldOutlineStyles, { moduleId: 'vaadin-f
  * @mixes ThemableMixin
  * @private
  */
-export class FieldOutline extends ThemableMixin(DirMixin(PolymerElement)) {
+export class FieldOutline extends ThemableMixin(DirMixin(PolylitMixin(LitElement))) {
   static get is() {
     return 'vaadin-field-outline';
   }
 
-  static get template() {
-    return html``;
-  }
-
   static get properties() {
     return {
       /**
@@ -37,10 +34,16 @@ export class FieldOutline extends ThemableMixin(DirMixin(PolymerElement)) {
         type: Object,
         value: null,
         observer: '_userChanged',
+        sync: true,
       },
     };
   }
 
+  /** @protected */
+  render() {
+    return html``;
+  }
+
   /** @protected */
   ready() {
     super.ready();

package/src/vaadin-user-tag.js

@@ -3,14 +3,13 @@
  * Copyright (c) 2021 - 2025 Vaadin Ltd.
  * This program is available under Apache License Version 2.0, available at https://vaadin.com/license/
  */
-import { html, PolymerElement } from '@polymer/polymer/polymer-element.js';
+import { html, LitElement } from 'lit';
 import { defineCustomElement } from '@vaadin/component-base/src/define.js';
 import { DirMixin } from '@vaadin/component-base/src/dir-mixin.js';
-import { registerStyles, ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mixin.js';
+import { PolylitMixin } from '@vaadin/component-base/src/polylit-mixin.js';
+import { ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mixin.js';
 import { userTagStyles } from './vaadin-field-highlighter-styles.js';
 
-registerStyles('vaadin-user-tag', userTagStyles, { moduleId: 'vaadin-user-tag-styles' });
-
 /**
  * An element used internally by `<vaadin-field-highlighter>`. Not intended to be used separately.
  *
@@ -20,13 +19,18 @@ registerStyles('vaadin-user-tag', userTagStyles, { moduleId: 'vaadin-user-tag-st
  * @mixes ThemableMixin
  * @private
  */
-export class UserTag extends ThemableMixin(DirMixin(PolymerElement)) {
+export class UserTag extends ThemableMixin(DirMixin(PolylitMixin(LitElement))) {
   static get is() {
     return 'vaadin-user-tag';
   }
 
-  static get template() {
-    return html`<div part="name">[[name]]</div>`;
+  static get styles() {
+    return userTagStyles;
+  }
+
+  /** @protected */
+  render() {
+    return html`<div part="name">${this.name}</div>`;
   }
 
   static get properties() {

package/src/vaadin-user-tags-overlay.js

@@ -3,17 +3,16 @@
  * Copyright (c) 2021 - 2025 Vaadin Ltd.
  * This program is available under Apache License Version 2.0, available at https://vaadin.com/license/
  */
-import { html, PolymerElement } from '@polymer/polymer/polymer-element.js';
+import { html, LitElement } from 'lit';
 import { defineCustomElement } from '@vaadin/component-base/src/define.js';
 import { DirMixin } from '@vaadin/component-base/src/dir-mixin.js';
+import { PolylitMixin } from '@vaadin/component-base/src/polylit-mixin.js';
 import { OverlayMixin } from '@vaadin/overlay/src/vaadin-overlay-mixin.js';
 import { PositionMixin } from '@vaadin/overlay/src/vaadin-overlay-position-mixin.js';
 import { overlayStyles } from '@vaadin/overlay/src/vaadin-overlay-styles.js';
-import { registerStyles, ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mixin.js';
+import { ThemableMixin } from '@vaadin/vaadin-themable-mixin/vaadin-themable-mixin.js';
 import { userTagsOverlayStyles } from './vaadin-field-highlighter-styles.js';
 
-registerStyles('vaadin-user-tags-overlay', [overlayStyles, userTagsOverlayStyles]);
-
 /**
  * An element used internally by `<vaadin-field-highlighter>`. Not intended to be used separately.
  *
@@ -25,14 +24,19 @@ registerStyles('vaadin-user-tags-overlay', [overlayStyles, userTagsOverlayStyles
  * @mixes ThemableMixin
  * @private
  */
-class UserTagsOverlay extends PositionMixin(OverlayMixin(DirMixin(ThemableMixin(PolymerElement)))) {
+class UserTagsOverlay extends PositionMixin(OverlayMixin(DirMixin(ThemableMixin(PolylitMixin(LitElement))))) {
   static get is() {
     return 'vaadin-user-tags-overlay';
   }
 
-  static get template() {
+  static get styles() {
+    return [overlayStyles, userTagsOverlayStyles];
+  }
+
+  /** @protected */
+  render() {
     return html`
-      <div id="backdrop" part="backdrop" hidden$="[[!withBackdrop]]"></div>
+      <div id="backdrop" part="backdrop" ?hidden="${!this.withBackdrop}"></div>
       <div part="overlay" id="overlay">
         <div part="content" id="content">
           <slot></slot>

package/src/vaadin-user-tags.js

@@ -5,11 +5,11 @@
  */
 import './vaadin-user-tag.js';
 import './vaadin-user-tags-overlay.js';
-import { calculateSplices } from '@polymer/polymer/lib/utils/array-splice.js';
-import { html, PolymerElement } from '@polymer/polymer/polymer-element.js';
+import { css, html, LitElement } from 'lit';
 import { timeOut } from '@vaadin/component-base/src/async.js';
 import { Debouncer } from '@vaadin/component-base/src/debounce.js';
 import { defineCustomElement } from '@vaadin/component-base/src/define.js';
+import { PolylitMixin } from '@vaadin/component-base/src/polylit-mixin.js';
 
 const listenOnce = (elem, type) => {
   return new Promise((resolve) => {
@@ -28,24 +28,28 @@ const listenOnce = (elem, type) => {
  * @extends HTMLElement
  * @private
  */
-export class UserTags extends PolymerElement {
+export class UserTags extends PolylitMixin(LitElement) {
   static get is() {
     return 'vaadin-user-tags';
   }
 
-  static get template() {
+  static get styles() {
+    return css`
+      :host {
+        position: absolute;
+      }
+    `;
+  }
+
+  /** @protected */
+  render() {
     return html`
-      <style>
-        :host {
-          position: absolute;
-        }
-      </style>
       <vaadin-user-tags-overlay
         id="overlay"
         modeless
-        opened="[[opened]]"
+        .opened="${this.opened}"
         no-vertical-overlap
-        on-vaadin-overlay-open="_onOverlayOpen"
+        @vaadin-overlay-open="${this._onOverlayOpen}"
       ></vaadin-user-tags-overlay>
     `;
   }
@@ -68,6 +72,7 @@ export class UserTags extends PolymerElement {
       opened: {
         type: Boolean,
         value: false,
+        sync: true,
       },
 
       /**
@@ -233,27 +238,6 @@ export class UserTags extends PolymerElement {
     return { added, removed };
   }
 
-  getChangedUsers(users, splices) {
-    const usersToAdd = [];
-    const usersToRemove = [];
-
-    splices.forEach((splice) => {
-      splice.removed.forEach((user) => {
-        usersToRemove.push(user);
-      });
-
-      for (let i = splice.addedCount - 1; i >= 0; i--) {
-        usersToAdd.push(users[splice.index + i]);
-      }
-    });
-
-    // Filter out users that are only moved
-    const addedUsers = usersToAdd.filter((u) => !usersToRemove.some((u2) => u.id === u2.id));
-    const removedUsers = usersToRemove.filter((u) => !usersToAdd.some((u2) => u.id === u2.id));
-
-    return { addedUsers, removedUsers };
-  }
-
   applyTagsStart({ added, removed }) {
     const wrapper = this.wrapper;
     removed.forEach((tag) => {
@@ -279,12 +263,22 @@ export class UserTags extends PolymerElement {
     // Apply pending change if needed
     this.requestContentUpdate();
 
-    const splices = calculateSplices(users, this.users);
-    if (splices.length === 0) {
-      return;
+    let addedUsers = [];
+    let removedUsers = [];
+
+    const hasNewUsers = Array.isArray(users);
+    const hasOldUsers = Array.isArray(this.users);
+
+    if (hasOldUsers) {
+      const newUserIds = (users || []).map((user) => user.id);
+      removedUsers = this.users.filter((item) => !newUserIds.includes(item.id));
+    }
+
+    if (hasNewUsers) {
+      const oldUserIds = (this.users || []).map((user) => user.id);
+      addedUsers = users.filter((item) => !oldUserIds.includes(item.id)).reverse();
     }
 
-    const { addedUsers, removedUsers } = this.getChangedUsers(users, splices);
     if (addedUsers.length === 0 && removedUsers.length === 0) {
       return;
     }
@@ -300,7 +294,7 @@ export class UserTags extends PolymerElement {
 
         this.__flashQueue.forEach((tags) => {
           if (tags.some((tag) => tag.uid === user.id)) {
-            this.splice('__flashQueue', i, 1);
+            this.__flashQueue = this.__flashQueue.filter((_, index) => index !== i);
           }
         });
       });
@@ -323,7 +317,7 @@ export class UserTags extends PolymerElement {
 
       if (this.flashing || !this.__isTargetVisible) {
         // Schedule next flash later
-        this.push('__flashQueue', addedTags);
+        this.__flashQueue = [...this.__flashQueue, addedTags];
       } else {
         this.flashTags(addedTags);
       }
@@ -394,7 +388,7 @@ export class UserTags extends PolymerElement {
     }).then(() => {
       if (this.__flashQueue.length > 0) {
         const tags = this.__flashQueue[0];
-        this.splice('__flashQueue', 0, 1);
+        this.__flashQueue = [...this.__flashQueue].slice(1);
         this.flashTags(tags);
       }
     });
@@ -416,7 +410,7 @@ export class UserTags extends PolymerElement {
     this.applyTagsStart(changed);
 
     this._debounceRender = Debouncer.debounce(this._debounceRender, timeOut.after(this.duration), () => {
-      this.set('users', users);
+      this.users = users;
 
       this.applyTagsEnd(changed);
 
@@ -428,7 +422,7 @@ export class UserTags extends PolymerElement {
 
   updateTagsSync(users, changed) {
     this.applyTagsStart(changed);
-    this.set('users', users);
+    this.users = users;
     this.applyTagsEnd(changed);
   }