Merge pull request #8956 from anntzer/text.set(bbox=None)

dopplershift · web-flow · commit 86bca7a3f9ae · 2017-08-24T14:04:18.000-06:00
Fix `text.set(bbox=None)`.
diff --git a/lib/matplotlib/text.py b/lib/matplotlib/text.py
@@ -238,10 +238,12 @@ def update(self, kwargs):
         """
         Update properties from a dictionary.
         """
-        bbox = kwargs.pop('bbox', None)
+        # Update bbox last, as it depends on font properties.
+        sentinel = object()  # bbox can be None, so use another sentinel.
+        bbox = kwargs.pop("bbox", sentinel)
         super(Text, self).update(kwargs)
-        if bbox:
-            self.set_bbox(bbox)  # depends on font properties
+        if bbox is not sentinel:
+            self.set_bbox(bbox)
 
     def __getstate__(self):
         d = super(Text, self).__getstate__()
