Skip to content

Commit 86bca7a

Browse files
authored
Merge pull request #8956 from anntzer/text.set(bbox=None)
Fix `text.set(bbox=None)`.
2 parents bcc2f12 + 2ee109d commit 86bca7a

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

lib/matplotlib/text.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -238,10 +238,12 @@ def update(self, kwargs):
238238
"""
239239
Update properties from a dictionary.
240240
"""
241-
bbox = kwargs.pop('bbox', None)
241+
# Update bbox last, as it depends on font properties.
242+
sentinel = object() # bbox can be None, so use another sentinel.
243+
bbox = kwargs.pop("bbox", sentinel)
242244
super(Text, self).update(kwargs)
243-
if bbox:
244-
self.set_bbox(bbox) # depends on font properties
245+
if bbox is not sentinel:
246+
self.set_bbox(bbox)
245247

246248
def __getstate__(self):
247249
d = super(Text, self).__getstate__()

0 commit comments

Comments
 (0)