FIX: make properties

jklymak · jklymak · commit 28621eedc107 · 2021-12-24T09:59:10.000+01:00
diff --git a/lib/matplotlib/figure.py b/lib/matplotlib/figure.py
@@ -1142,7 +1142,7 @@ def colorbar(
                     "to colorbar().")
 
         if (self.get_layout_engine() is not None and
-                not self.get_layout_engine().get_colorbar_gridspec()):
+                not self.get_layout_engine().colorbar_gridspec):
             use_gridspec = False
         # Store the value of gca so that we can set it back later on.
         if cax is None:
@@ -1197,7 +1197,7 @@ def subplots_adjust(self, left=None, bottom=None, right=None, top=None,
             as a fraction of the average Axes height.
         """
         if (self.get_layout_engine() is not None and
-                not self.get_layout_engine().get_adjust_compatible()):
+                not self.get_layout_engine().adjust_compatible):
             _api.warn_external(
                 "This figure was using a layout engine that is "
                 "incompatible with subplots_adjust and/or tight_layout; "
diff --git a/lib/matplotlib/layout_engine.py b/lib/matplotlib/layout_engine.py
@@ -3,15 +3,15 @@
 
 Figures have a ``layout_engine`` property that holds a subclass of
 `~.LayoutEngine` defined here (or *None* for no layout).  At draw time
-``figure.get_layout_engine().execute()`` is called, the goal of which is usually
-to rearrange Axes on the figure to produce a pleasing layout. This is like a
-``draw`` callback, however when printing we disable the layout engine for the
-final draw and it is useful to know the layout engine while the figure is being
-created, in particular to deal with colorbars.
+``figure.get_layout_engine().execute()`` is called, the goal of which is
+usually to rearrange Axes on the figure to produce a pleasing layout. This is
+like a ``draw`` callback, however when printing we disable the layout engine
+for the final draw and it is useful to know the layout engine while the figure
+is being created, in particular to deal with colorbars.
 
 Matplotlib supplies two layout engines, `.TightLayoutEngine` and
-`.ConstrainedLayoutEngine`.  Third parties can create their own layout engine by
-subclassing `.LayoutEngine`.
+`.ConstrainedLayoutEngine`.  Third parties can create their own layout engine
+by subclassing `.LayoutEngine`.
 """
 
 from contextlib import nullcontext
@@ -39,7 +39,7 @@ class LayoutEngine:
     at draw time by `~.figure.Figure.draw`, providing a special draw-time hook.
 
     Currently, there are two properties of ``LayoutEngine`` classes that are
-    consulted while manipulating the figure.  ``engine.get_colorbar_gridspec``
+    consulted while manipulating the figure.  ``engine.colorbar_gridspec``
     tells `.Figure.colorbar` whether to make the axes using the gridspec
     method (see `.colorbar.make_axes_gridspec`) or not
     (see `.colorbar.make_axes`); `.ConstrainedLayoutEngine` sets this to
@@ -58,14 +58,16 @@ def __init__(self):
     def set(self, **kwargs):
         raise NotImplementedError
 
-    def get_colorbar_gridspec(self):
+    @property
+    def colorbar_gridspec(self):
         """
         Return a boolean if the layout engine creates colorbars using a
         gridspec.
         """
         return self._colorbar_gridspec
 
-    def get_adjust_compatible(self):
+    @property
+    def adjust_compatible(self):
         """
         Return a boolean if the layout engine is compatible with
         `~.Figure.subplots_adjust`.
@@ -163,6 +165,7 @@ def set(self, *, pad=None, w_pad=None, h_pad=None, rect=None):
             if locals()[td] is not None:
                 self._params[td] = locals()[td]
 
+
 class ConstrainedLayoutEngine(LayoutEngine):
     """
     Implements the ``constrained_layout`` geometry management.  See
