Antony Lee

The error doesn't even have to be confusing, we can word it appropriately. I'd just put in a normal deprecation warning and hopefully someone will complain if they're not happy...

Having None mean "the default" is useful when that default is not easily specifiable otherwise (e.g., when it is (derived from) an rcParam). When the default is IdentityTransform(), I don't...

Curiously, I get similar result with mplcairo: chromium shows the hatches, mupdf and zathura don't, and okular only shows the round ones (second from the left).

Previously ticks whose label1On attribute was set to False were excluded (per https://github.com/matplotlib/matplotlib/commit/525dcef84a4da2383230d49a0689c66ee49aa3b6#diff-52127080e2783464f529df684d82d476L1239), so I just remapped the behavior to get_visible(), as that was the point of #10088. So it's...

If we are going to change this (unclear whether we do) I would rather always return all of them and let the user filter them as needed -- we don't...

Sorry, I think I'm trying to do too much here, let me split that PR into smaller ones...

Feel free to pick up whatever is relevant from my patch and either close this if you think everything that matters has been handled, or leave it open if there's...

Just a quick drive-by comment: there's already `ax.set_navigate(False)`, which can disable pan/zoom on an Axes.

I can repro the issue as of master (also in a different context, specifically when a cppimported module (https://github.com/tbenthompson/cppimport) fails to compile. The following simpler patch seems to fix the...

I agree the API is pretty confusing, but that's what it was before. Still, it can be improved (last commit now) by making the callables passed to set_offset take no...