@ni/nimble-components 24.1.2 → 24.1.4

package/dist/all-components-bundle.js

@@ -16301,7 +16301,7 @@
 
   /**
    * Do not edit directly
-   * Generated on Thu, 28 Mar 2024 17:10:03 GMT
+   * Generated on Tue, 02 Apr 2024 16:52:55 GMT
    */
 
   const Information100DarkUi = "#a46eff";
@@ -21202,6 +21202,21 @@
           this.emitChangeIfValueUpdated();
           return returnValue;
       }
+      /**
+       * @internal
+       */
+      registerOption(option) {
+          if (this.options.includes(option)) {
+              return;
+          }
+          // Adding an option to the end, ultimately, isn't the correct
+          // thing to do, as this will mean the option's index in the options,
+          // at least temporarily, does not match the DOM order. However, it
+          // is expected that a successive run of `slottedOptionsChanged` will
+          // correct this order issue. See 'https://github.com/ni/nimble/issues/1915'
+          // for more info.
+          this.options.push(option);
+      }
       focusAndScrollOptionIntoView() {
           if (this.open) {
               super.focusAndScrollOptionIntoView();
@@ -21853,6 +21868,12 @@
           this.triggerAnimation();
       }
       triggerAnimation() {
+          // Read the offsetHeight of the dialog to trigger a reflow. This guarantees that the browser
+          // has processed the 'animating' class being removed before trying to readd it, even if the previous
+          // animation has just finished. Otherwise, problems can occur. For example, trying to close the
+          // drawer immediately after the opening animation ends does not actually close the drawer.
+          // https://github.com/ni/nimble/issues/1994
+          void this.dialog.offsetHeight;
           this.dialog.classList.add('animating');
           if (this.closing) {
               this.dialog.classList.add('closing');