Skip to content

Sty solarized #9026

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
Aug 17, 2017
Merged

Sty solarized #9026

merged 4 commits into from
Aug 17, 2017

Conversation

tacaswell
Copy link
Member

Rebase of #3851

PR Summary

PR Checklist

  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant

ilivni and others added 2 commits August 12, 2017 21:22
Adapted from ggplot2

Title padding
Title color
Minor grid width
Alpha for graphs with a "fill"

Solarized_License

Taken from the Solarized Github page:
https://github.com/altercation/solarized

Example Script

No changes to the mplstyle.

Changed Name of File

File Conventions

Conforming to matplolib conventions

Deleted Some Empty Lines

Failed last  Travis regression

Get Rid of white spaces 2

Solaraized Light Style File Fixed

style sheet remove white spaces

Tried setting the user save preference
"trim_trailing_white_space_on_save": true

3rd Try White Spaces

Used convert tabs -> spaces script

Used PSPAD to check whitespace

Set Sublime to UNIX Line style endings

Tried Unix style line endings with sublime
@tacaswell tacaswell added this to the 2.1 (next point release) milestone Aug 13, 2017
@tacaswell tacaswell mentioned this pull request Aug 13, 2017
@dopplershift dopplershift changed the title Sty solaized Sty solarized Aug 14, 2017
@tacaswell
Copy link
Member Author

@dopplershift
Copy link
Contributor

@choldgraf choldgraf merged commit 5044788 into matplotlib:master Aug 17, 2017
@QuLogic
Copy link
Member

QuLogic commented Aug 17, 2017

I still don't understand why there's a 2 in the name.

@tacaswell tacaswell deleted the sty_solaized branch August 27, 2017 18:54
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.

5 participants