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

Jump to bottom

fixes #569 #570

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fixes #569 #570

wants to merge 1 commit into from

Conversation

ceremcem
Copy link

@ceremcem ceremcem commented Jul 8, 2017

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 50.778% when pulling 8f177df on ceremcem:patch-1 into 6e2b7a7 on shutterstock:master.

@RichardLitt
Copy link
Contributor

RichardLitt commented Aug 25, 2017 •
edited

This doesn't work as expected. I'm not seeing the x_label at all. bottom: 40px; seems to remove it from the graph.

screen shot 2017-08-25 at 4 13 06 pm

These edits should also be in css/detail.css, as the Makefile compiles them from there, and they're referenced in those files in the examples.

@ceremcem Can you move the CSS to the proper location, and then fix the issue?

@ceremcem
Copy link
Author

ceremcem commented Aug 25, 2017 •
edited

Yes, I knew that it should also be defined in somewhere else :) Okay, it's on me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
waiting-on-contributor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ceremcem @coveralls @RichardLitt