Skip to content
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

using theme.grid_strokedash #339

Merged
merged 4 commits into from
Jun 28, 2014
Merged

using theme.grid_strokedash #339

merged 4 commits into from
Jun 28, 2014

Conversation

inq
Copy link
Contributor

@inq inq commented Jun 27, 2014

I want to use solid lines for the grid, but it seems to be static.
Fortunately, I found the corresponding variable in the theme.
So I modified the part to use the given variable (theme.grid_strokedash).

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 2360a90 on inq:master into 30bd786 on dcjones:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-16.09%) when pulling 45f67d4 on inq:master into 30bd786 on dcjones:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-16.76%) when pulling 45f67d4 on inq:master into 30bd786 on dcjones:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling a5a725b on inq:master into 30bd786 on dcjones:master.

dcjones added a commit that referenced this pull request Jun 28, 2014
using theme.grid_strokedash
@dcjones dcjones merged commit a5e5d9e into GiovineItalia:master Jun 28, 2014
@dcjones
Copy link
Collaborator

dcjones commented Jun 28, 2014

Thanks, that definitely shouldn't have been hard-coded. You attached some more commits after the initial PR. Those look good too, but in the future please split multiple changes into multiple pull requests.

dcjones added a commit that referenced this pull request Jun 28, 2014
dcjones added a commit that referenced this pull request Jun 28, 2014
@inq
Copy link
Contributor Author

inq commented Jun 29, 2014

Oops! I'm sorry
My personal trial was attached at the same time!
I'll care them later. Thank you!

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.

3 participants