Skip to content

Conversation

@darribas
Copy link
Collaborator

@darribas darribas commented Aug 1, 2019

This provides a hack to wrap the text of the attribution.

@coveralls
Copy link

coveralls commented Aug 1, 2019

Coverage Status

Coverage remained the same at 92.711% when pulling 66ae705 on text_wrap#93 into c7e3217 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.06%) to 92.775% when pulling d508c98 on text_wrap#93 into c7e3217 on master.

@jorisvandenbossche jorisvandenbossche changed the title Text wrap#93 ENH: wrap attribution text Aug 1, 2019
)
# hack to have the text wrapped in the ax extent, for some explanation see
# https://stackoverflow.com/questions/48079364/wrapping-text-not-working-in-matplotlib
wrap_width = ax.get_window_extent().width * 0.99
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I changed this from 0.9 to 0.99 so we have the same margin left and right (0.005 at both sides -> remaining text width of 0.99)

@jorisvandenbossche
Copy link
Member

Did some more testing, this seems to work nicely!

@jorisvandenbossche jorisvandenbossche merged commit 30161bf into master Aug 1, 2019
@jorisvandenbossche jorisvandenbossche deleted the text_wrap#93 branch August 1, 2019 18:44
This was referenced Aug 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants