@fluentui/web-components 2.5.7 → 2.5.8

package/CHANGELOG.json

@@ -2,7 +2,22 @@
   "name": "@fluentui/web-components",
   "entries": [
     {
-      "date": "Fri, 21 Oct 2022 07:43:28 GMT",
+      "date": "Wed, 26 Oct 2022 07:51:18 GMT",
+      "tag": "@fluentui/web-components_v2.5.8",
+      "version": "2.5.8",
+      "comments": {
+        "patch": [
+          {
+            "author": "chhol@microsoft.com",
+            "package": "@fluentui/web-components",
+            "commit": "400b81dc432ff1720a89ffafee0666c4e920272b",
+            "comment": "emit activeid as part of accordion change event"
+          }
+        ]
+      }
+    },
+    {
+      "date": "Fri, 21 Oct 2022 07:44:53 GMT",
       "tag": "@fluentui/web-components_v2.5.7",
       "version": "2.5.7",
       "comments": {

package/CHANGELOG.md

@@ -1,12 +1,21 @@
 # Change Log - @fluentui/web-components
 
-This log was last generated on Fri, 21 Oct 2022 07:43:28 GMT and should not be manually modified.
+This log was last generated on Wed, 26 Oct 2022 07:51:18 GMT and should not be manually modified.
 
 <!-- Start content -->
 
+## [2.5.8](https://github.com/microsoft/fluentui/tree/@fluentui/web-components_v2.5.8)
+
+Wed, 26 Oct 2022 07:51:18 GMT 
+[Compare changes](https://github.com/microsoft/fluentui/compare/@fluentui/web-components_v2.5.7..@fluentui/web-components_v2.5.8)
+
+### Patches
+
+- emit activeid as part of accordion change event ([PR #25364](https://github.com/microsoft/fluentui/pull/25364) by chhol@microsoft.com)
+
 ## [2.5.7](https://github.com/microsoft/fluentui/tree/@fluentui/web-components_v2.5.7)
 
-Fri, 21 Oct 2022 07:43:28 GMT 
+Fri, 21 Oct 2022 07:44:53 GMT 
 [Compare changes](https://github.com/microsoft/fluentui/compare/@fluentui/web-components_v2.5.6..@fluentui/web-components_v2.5.7)
 
 ### Patches

package/dist/web-components.js

@@ -86,6 +86,32 @@ if (FAST.getById === void 0) {
 
 
 const emptyArray = Object.freeze([]);
+/**
+ * Creates a function capable of locating metadata associated with a type.
+ * @returns A metadata locator function.
+ * @internal
+ */
+
+function createMetadataLocator() {
+  const metadataLookup = new WeakMap();
+  return function (target) {
+    let metadata = metadataLookup.get(target);
+
+    if (metadata === void 0) {
+      let currentTarget = Reflect.getPrototypeOf(target);
+
+      while (metadata === void 0 && currentTarget !== null) {
+        metadata = metadataLookup.get(currentTarget);
+        currentTarget = Reflect.getPrototypeOf(currentTarget);
+      }
+
+      metadata = metadata === void 0 ? [] : metadata.slice(0);
+      metadataLookup.set(target, metadata);
+    }
+
+    return metadata;
+  };
+}
 
 const updateQueue = $global.FAST.getById(1
 /* updateQueue */
@@ -530,7 +556,6 @@ const Observable = FAST.getById(2
 , () => {
   const volatileRegex = /(:|&&|\|\||if)/;
   const notifierLookup = new WeakMap();
-  const accessorLookup = new WeakMap();
   const queueUpdate = DOM.queueUpdate;
   let watcher = void 0;
 
@@ -552,28 +577,7 @@ const Observable = FAST.getById(2
     return found;
   }
 
-  function getAccessors(target) {
-    let accessors = accessorLookup.get(target);
-
-    if (accessors === void 0) {
-      let currentTarget = Reflect.getPrototypeOf(target);
-
-      while (accessors === void 0 && currentTarget !== null) {
-        accessors = accessorLookup.get(currentTarget);
-        currentTarget = Reflect.getPrototypeOf(currentTarget);
-      }
-
-      if (accessors === void 0) {
-        accessors = [];
-      } else {
-        accessors = accessors.slice(0);
-      }
-
-      accessorLookup.set(target, accessors);
-    }
-
-    return accessors;
-  }
+  const getAccessors = createMetadataLocator();
 
   class DefaultObservableAccessor {
     constructor(name) {
@@ -1609,8 +1613,9 @@ class HTMLView {
     if (this.fragment.hasChildNodes()) {
       node.parentNode.insertBefore(this.fragment, node);
     } else {
-      const parentNode = node.parentNode;
       const end = this.lastChild;
+      if (node.previousSibling === end) return;
+      const parentNode = node.parentNode;
       let current = this.firstChild;
       let next;
 
@@ -2096,6 +2101,17 @@ class StyleElementStyles extends ElementStyles {
 
 }
 
+/**
+ * Metadata used to configure a custom attribute's behavior.
+ * @public
+ */
+
+const AttributeConfiguration = Object.freeze({
+  /**
+   * Locates all attribute configurations associated with a type.
+   */
+  locate: createMetadataLocator()
+});
 /**
  * A {@link ValueConverter} that converts to and from `boolean` values.
  * @remarks
@@ -2262,7 +2278,7 @@ class AttributeDefinition {
 
   static collect(Owner, ...attributeLists) {
     const attributes = [];
-    attributeLists.push(Owner.attributes);
+    attributeLists.push(AttributeConfiguration.locate(Owner));
 
     for (let i = 0, ii = attributeLists.length; i < ii; ++i) {
       const list = attributeLists[i];
@@ -2299,8 +2315,7 @@ function attr(configOrTarget, prop) {
       config.property = $prop;
     }
 
-    const attributes = $target.constructor.attributes || ($target.constructor.attributes = []);
-    attributes.push(config);
+    AttributeConfiguration.locate($target.constructor).push(config);
   }
 
   if (arguments.length > 1) {
@@ -3876,7 +3891,7 @@ class RepeatBehavior {
       let addIndex = splice.index;
       const end = addIndex + splice.addedCount;
       const removedViews = views.splice(splice.index, removed.length);
-      availableViews = leftoverViews.length + removedViews.length;
+      const totalAvailableViews = availableViews = leftoverViews.length + removedViews.length;
 
       for (; addIndex < end; ++addIndex) {
         const neighbor = views[addIndex];
@@ -3884,7 +3899,7 @@ class RepeatBehavior {
         let view;
 
         if (recycle && availableViews > 0) {
-          if (removeIndex <= availableViews && removedViews.length > 0) {
+          if (removeIndex <= totalAvailableViews && removedViews.length > 0) {
             view = removedViews[removeIndex];
             removeIndex++;
           } else {
@@ -5946,7 +5961,9 @@ class FoundationElementRegistry {
  * Sourced from {@link https://www.typescriptlang.org/docs/handbook/mixins.html | TypeScript Documentation }.
  * @public
  */
+
 function applyMixins(derivedCtor, ...baseCtors) {
+  const derivedAttributes = AttributeConfiguration.locate(derivedCtor);
   baseCtors.forEach(baseCtor => {
     Object.getOwnPropertyNames(baseCtor.prototype).forEach(name => {
       if (name !== "constructor") {
@@ -5955,11 +5972,8 @@ function applyMixins(derivedCtor, ...baseCtors) {
         Object.getOwnPropertyDescriptor(baseCtor.prototype, name));
       }
     });
-
-    if (baseCtor.attributes) {
-      const existing = derivedCtor.attributes || [];
-      derivedCtor.attributes = existing.concat(baseCtor.attributes);
-    }
+    const baseAttributes = AttributeConfiguration.locate(baseCtor);
+    baseAttributes.forEach(x => derivedAttributes.push(x));
   });
 }
 
@@ -6391,7 +6405,7 @@ class Accordion extends FoundationElement {
     this.activeItemIndex = 0;
 
     this.change = () => {
-      this.$emit("change");
+      this.$emit("change", this.activeid);
     };
 
     this.setItems = () => {