Skip to content

Conversation

@tacaswell
Copy link
Member

Use set_tick_params to hide tick labels in not-edge plots instead
of setting the visibility on the tick label objects.

This catches both major and minor tick-labels and is more robust to
changes in the ticklabel generation.

closes #8903

PR Summary

PR Checklist

  • Has Pytest style unit tests
  • Code is PEP 8 compliant

@tacaswell tacaswell added this to the 2.1.1 (next bug fix release) milestone Aug 6, 2017
Use `set_tick_params` to hide tick labels in not-edge plots instead
of setting the visibility on the tick label objects.

This catches both major and minor tick-labels and is more robust to
changes in the ticklabel generation.

closes matplotlib#8903
@tacaswell tacaswell force-pushed the fix_sharex_log_minorlabels branch from 7c4eaea to 8fd417e Compare August 6, 2017 22:54
@tacaswell
Copy link
Member Author

In which I learn that I did not have flake8 installed in the current venv....

@QuLogic
Copy link
Member

QuLogic commented Aug 7, 2017

I think this change might be part of #8678, though of course not with this intention.

@tacaswell
Copy link
Member Author

Ah, I thought #8678 was already merged :( That needs a rebase anyway, can you fold this commit into that PR?

@tacaswell tacaswell closed this Aug 13, 2017
@tacaswell tacaswell deleted the fix_sharex_log_minorlabels branch August 13, 2017 02:17
@QuLogic QuLogic modified the milestones: 2.1 (next point release), 2.1.1 (next bug fix release) Aug 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Minor tick labels shown on all axes with log scale and share{x,y}=True

2 participants