Skip to content

BUG: Fixed font size (set it on both x and y axis). #8765 #8940

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

dxe4
Copy link
Contributor

@dxe4 dxe4 commented Nov 30, 2014

Fix for #8765
The font size was only set on x axis if vertical or the y axis if horizontal.

@jorisvandenbossche
Copy link
Member

@TomAugspurger can you review this?

@jorisvandenbossche jorisvandenbossche added this to the 0.15.2 milestone Nov 30, 2014
@dxe4 dxe4 force-pushed the 8765_bug_fontsize_only_on_x_axis branch from 826f862 to c76ba87 Compare November 30, 2014 13:39
@TomAugspurger
Copy link
Contributor

@papaloizouc ready to merge if you could add a short note to doc/source/whatsnew/v0.15.2.

@@ -48,6 +48,14 @@ def test_ts_plot_with_tz(self):
ts = Series([188.5, 328.25], index=index)
_check_plot_works(ts.plot)

def test_fontsize_set_correctly(self):
'''For issue #8765'''
Copy link
Member

Choose a reason for hiding this comment

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

Can you make this a # comment (docstring disguise the test name when running the tests, which is not really handy)

@dxe4 dxe4 force-pushed the 8765_bug_fontsize_only_on_x_axis branch 2 times, most recently from 6d62a66 to 9f093c7 Compare November 30, 2014 15:55
@dxe4 dxe4 force-pushed the 8765_bug_fontsize_only_on_x_axis branch from 9f093c7 to 47a831e Compare November 30, 2014 15:58
@jorisvandenbossche
Copy link
Member

Thanks!

jorisvandenbossche added a commit that referenced this pull request Dec 1, 2014
…_axis

BUG: Fixed font size (set it on both x and y axis). #8765
@jorisvandenbossche jorisvandenbossche merged commit 2ad9dc7 into pandas-dev:master Dec 1, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants