@douyinfe/semi-ui 2.43.0-beta.0 → 2.43.0

package/dist/umd/semi-ui.js

@@ -39223,10 +39223,15 @@ class TagInputFoundation extends foundation {
       }
     };
     this.handleInputCompositionStart = e => {
+      const {
+        maxLength
+      } = this.getProps();
+      if (!isNumber_default()(maxLength)) {
+        return;
+      }
       this._adapter.setEntering(true);
     };
     this.handleInputCompositionEnd = e => {
-      this._adapter.setEntering(false);
       const {
         value
       } = e.target;
@@ -39235,27 +39240,31 @@ class TagInputFoundation extends foundation {
         onInputExceed,
         separator
       } = this.getProps();
+      if (!isNumber_default()(maxLength)) {
+        return;
+      }
+      this._adapter.setEntering(false);
       let allowChange = true;
-      const {
-        inputValue
-      } = this.getStates();
-      if (isNumber_default()(maxLength)) {
-        const inputArr = utils_getSplitedArray(inputValue, separator);
-        let index = 0;
-        for (; index < inputArr.length; index++) {
-          if (inputArr[index].length > maxLength) {
-            allowChange = false;
-            isFunction_default()(onInputExceed) && onInputExceed(value);
-            break;
-          }
+      const inputArr = utils_getSplitedArray(value, separator);
+      let index = 0;
+      for (; index < inputArr.length; index++) {
+        if (inputArr[index].length > maxLength) {
+          allowChange = false;
+          isFunction_default()(onInputExceed) && onInputExceed(value);
+          break;
         }
-        if (!allowChange) {
-          const newInputArr = inputArr.slice(0, index);
-          if (index < inputArr.length) {
-            newInputArr.push(inputArr[index].slice(0, maxLength));
-          }
-          this._adapter.setInputValue(newInputArr.join(separator));
+      }
+      if (!allowChange) {
+        const newInputArr = inputArr.slice(0, index);
+        if (index < inputArr.length) {
+          newInputArr.push(inputArr[index].slice(0, maxLength));
         }
+        this._adapter.setInputValue(newInputArr.join(separator));
+      } else {
+        // Why does it need to be updated here instead of in onChange when the value meets the maxLength limit?
+        // Because in firefox, the state change in InputCompositionEnd causes onChange to not be triggered after 
+        // the composition input completes input.
+        this._adapter.setInputValue(value);
       }
     };
     /**
@@ -100796,8 +100805,8 @@ class Preview extends BaseComponent {
             if (item.isIntersecting && src) {
               item.target.src = src;
               item.target.removeAttribute("data-src");
+              this.previewObserver.unobserve(item.target);
             }
-            this.previewObserver.unobserve(item.target);
           });
         }, {
           root: document.querySelector(`#${this.previewGroupId}`),