@ni/nimble-components 24.1.3 → 24.1.5

package/dist/all-components-bundle.js

@@ -16301,7 +16301,7 @@
 
   /**
    * Do not edit directly
-   * Generated on Tue, 02 Apr 2024 15:16:13 GMT
+   * Generated on Tue, 02 Apr 2024 21:45:54 GMT
    */
 
   const Information100DarkUi = "#a46eff";
@@ -21202,21 +21202,6 @@
           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();
@@ -21868,6 +21853,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');