Skip to content

Small (but important) improvement to some themes #934

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
merged 4 commits into from
Mar 21, 2014

Conversation

jiho
Copy link
Contributor

@jiho jiho commented Mar 21, 2014

No description provided.

jiho added 2 commits March 21, 2014 17:09
Make the border thinner, to match the thin border of the panel itself. The
border of the panel is thin because it is clipped so it is not possible to
match it exactly but this is already an improvement over the previous version.
Facet strips were filled and legend keys were grey while all the rest is
black and white and as minimal as possible. Changes are made to match that
feel.
@jiho
Copy link
Contributor Author

jiho commented Mar 21, 2014

Cosmetic tweaks to theme_bw() and theme_classic().

I haven't removed the grid in theme_minimal() as initially planned because it might break people expectation too much...

@jiho jiho closed this Mar 21, 2014
@jiho
Copy link
Contributor Author

jiho commented Mar 21, 2014

No idea why it is closed... Reopening

@jiho jiho reopened this Mar 21, 2014
@hadley
Copy link
Member

hadley commented Mar 21, 2014

Would you mind generating a couple of before and after shots for me?

@jiho
Copy link
Contributor Author

jiho commented Mar 21, 2014

Before
image
After
image

Before
image
After
image

@hadley
Copy link
Member

hadley commented Mar 21, 2014

Can you mention this PR number in the news?

@jiho
Copy link
Contributor Author

jiho commented Mar 21, 2014

Done

PS: The copy-from-R-graphic-window and paste-inside-github functionality is fantastic!

hadley added a commit that referenced this pull request Mar 21, 2014
Small (but important) improvement to some themes
@hadley hadley merged commit 581fe35 into tidyverse:master Mar 21, 2014
@jiho jiho deleted the theme_improvements branch March 21, 2014 21:47
@lock
Copy link

lock bot commented Jan 19, 2019

This old issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with reprex) and link to this issue. https://reprex.tidyverse.org/

@lock lock bot locked and limited conversation to collaborators Jan 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants