Fix the UIOpacity issue when the node is re-added to the scene. (#17700)

* fix ui-opacity fix the render.node._uiProps.localOpacity only update on native. * Add JSB Check in _parentChanged function The setEntityLocalOpacityDirtyRecursively function should not work on web platform. * fix the opacity issue on native Whenever the node is re-added to the scene, the _parentOpacity value needs to be updated again in the onEnable callback. * Remove the JSB checking in setEntityLocalOpacityDirtyRecursively function * Update onEnable function
cocos · Oct 12, 2024 · 2ec9087 · 2ec9087
1 parent 186fa0f
commit 2ec9087
Showing 1 changed file with 14 additions and 2 deletions.
diff --git a/cocos/2d/components/ui-opacity.ts b/cocos/2d/components/ui-opacity.ts
@@ -59,6 +59,8 @@ export class UIOpacity extends Component {
      */
     private _parentOpacity: number = 1.0;
 
+    private _parentOpacityResetFlag: boolean = true;
+
     /**
      * @en
      * The transparency value of the impact.
@@ -97,7 +99,7 @@ export class UIOpacity extends Component {
             // }
             // UIRenderer.setEntityColorDirtyRecursively(this.node, dirty);
 
-            UIOpacity.setEntityLocalOpacityDirtyRecursively(this.node, dirty, 1, false);
+            UIOpacity.setEntityLocalOpacityDirtyRecursively(this.node, dirty, this._parentOpacity, false);
         }
     }
 
@@ -194,11 +196,16 @@ export class UIOpacity extends Component {
     }
 
     protected _parentChanged (): void {
+        if (!JSB) {
+            return;
+        }
         const parent = this.node.getParent();
         let opacity = 1;
         if (parent) {
             this._parentOpacity = this._getParentOpacity(parent);
             opacity = this._parentOpacity;
+        } else {
+            this._parentOpacityResetFlag = true;
         }
         UIOpacity.setEntityLocalOpacityDirtyRecursively(this.node, true, opacity, false);
     }
@@ -224,7 +231,12 @@ export class UIOpacity extends Component {
     public onEnable (): void {
         this.node.on(NodeEventType.PARENT_CHANGED, this._parentChanged, this);
         this.node._uiProps.localOpacity = this._parentOpacity * this._opacity / 255;
-        this._setEntityLocalOpacityRecursively(this.node._uiProps.localOpacity);
+        if (this._parentOpacityResetFlag) {
+            this._parentChanged();
+            this._parentOpacityResetFlag = false;
+        } else {
+            this._setEntityLocalOpacityRecursively(this.node._uiProps.localOpacity);
+        }
     }
 
     public onDisable (): void {