Skip to content

Commit 037e6ed

Browse files
committed
Move AxisArtistHelpers to toplevel.
The axisartist has a concept of "axis_artist_helper", which computes various computations to draw slanted/curved axises. Confusingly, `AxisArtistHelper` (and likewise `AxisArtistHelperRectlinear`) do *not* define such helper classes; they are simply namespaces that hold the `{AxisArtistHelper,AxisArtistHelperRectlinear}.{Fixed,Floating}` nested classes which *do* define helpers. More specifically, `AxisArtistHelper.{Fixed,Floating}` act as abstract base classes for `AxisArtistHelperRectlinear.{Fixed,Floating}` which are actually usable. In order to slightly disentangle this move the actual helper classes to the toplevel (as `_{Fixed,Floating}AxisArtistHelperBase` and `_{Fixed,Floating}AxisArtistHelperRectlinear`), keeping the old "purely namespace" classes around for backcompat. (But note that end users should never have to directly interact with these classes anyways -- normally, they only construct GridHelpers which take care of the interaction with AxisArtistHelpers; see e.g. the various axisartist examples.) More simply, this commit simply dedents most of the definitions of the Helper classes.
1 parent 6aaec5a commit 037e6ed

File tree

2 files changed

+201
-190
lines changed

2 files changed

+201
-190
lines changed

0 commit comments

Comments
 (0)