Preserve user-set patchA in Annotation.update_positions

dikshajangra12918-oss · web-flow · commit 2f804e7b538f · 2026-04-05T12:17:53.000+05:30
Add logic to prevent overriding user-set patchA in update_positions. Fixes issues #28316.
diff --git a/lib/matplotlib/text.py b/lib/matplotlib/text.py
@@ -2101,14 +2101,18 @@ def update_positions(self, renderer):
                 xy=(bbox.x0 - pad / 2, bbox.y0 - pad / 2),
                 width=bbox.width + pad, height=bbox.height + pad,
                 transform=IdentityTransform(), clip_on=False)
-        old_patchA = self.arrow_patch.patchA    
-        self.arrow_patch.set_patchA(patchA)
-
-        # Ensure arrow updates when patchA changes
-        if old_patchA is not patchA:
-            self.stale = True
-        
+            # Check if user manually set patchA externally
+            current_patchA = self.arrow_patch.patchA
+            internal_patchA = getattr(self, '_internal_patchA', None)
 
+            if current_patchA is not internal_patchA:
+                # patchA was manually set by the user, do not override it
+                pass
+            else:
+                # patchA was set internally, update it.
+                self.arrow_patch.set_patchA(patchA)
+                self._internal_patchA = patchA
+                
     @artist.allow_rasterization
     def draw(self, renderer):
         # docstring inherited
