@vaadin/field-highlighter 24.7.0-alpha5 → 24.7.0-alpha7

package/package.json

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

package/src/fields/outline.js

@@ -48,6 +48,10 @@ export const initOutline = (field) => {
 
     const style = document.createElement('style');
     style.textContent = `
+      :host([focused]) [part="outline"] {
+        opacity: calc(1 - var(--lumo-input-field-pointer-focus-visible, 0));
+      }
+
       :host([active]) [part="outline"],
       :host([focus-ring]) [part="outline"] {
         display: none;

package/src/vaadin-user-tags.js

@@ -5,7 +5,6 @@
  */
 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 { timeOut } from '@vaadin/component-base/src/async.js';
 import { Debouncer } from '@vaadin/component-base/src/debounce.js';
@@ -233,27 +232,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 +257,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;
     }