Merge pull request #14697 from timhoffm/fix-qt5-navigationtoolbar-size

tacaswell · web-flow · commit dcff0d27302a · 2019-07-05T12:14:00.000-04:00
FIX: NavigationToolbar2QT height
diff --git a/lib/matplotlib/backends/backend_qt5.py b/lib/matplotlib/backends/backend_qt5.py
@@ -727,14 +727,14 @@ def buttons(self):
     def adj_window(self):
         return None
 
-    if is_pyqt5():
-        # For some reason, self.setMinimumHeight doesn't seem to carry over to
-        # the actual sizeHint, so override it instead in order to make the
-        # aesthetic adjustments noted above.
-        def sizeHint(self):
-            size = super().sizeHint()
+    def sizeHint(self):
+        size = super().sizeHint()
+        if is_pyqt5() and self.canvas._dpi_ratio > 1:
+            # For some reason, self.setMinimumHeight doesn't seem to carry over
+            # to the actual sizeHint, so override it instead in order to make
+            # the aesthetic adjustments noted above.
             size.setHeight(max(48, size.height()))
-            return size
+        return size
 
     def edit_parameters(self):
         axes = self.canvas.figure.get_axes()
