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

Adding new function to move sliders from the outside #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lukius
Copy link

@lukius lukius commented Nov 20, 2013

I've just used d3.slider in an application to aid my students on an assignment. Since I was in the need of a way to make the slider move from the outside (that is, clicking buttons or pressing some keys), I decided to take a look at the code and hack it a little bit.

An example: http://lukius.github.io/d3.slider/

By the way, great job! Thanks!

@atharrison
Copy link

👍 This works where the current master branch of turban's appears not to, for me. Thanks go to both of you, for working on this!

@Seanny123
Copy link

FYI: the example website didn't work for me on Chrome.

@lukius
Copy link
Author

lukius commented Sep 24, 2014

@Seanny123: I've re-checked the website and it seems to be working fine under an up-to-date Chrome (as of today).

@Seanny123
Copy link

Maybe it's Ubuntu's fault? I'll try it later on Windows and I'll get back to you. Firefox works though. (:

@lukius
Copy link
Author

lukius commented Sep 25, 2014

Just tried on an up-to-date Chrome under Ubuntu 14.04 and works OK.

@Seanny123
Copy link

Oh jeez. It was one of my add-ons. I forgot to test opening the link in an incognito window. This is really embarrassing. Sorry about that.

@lukius
Copy link
Author

lukius commented Sep 25, 2014

No problem! Thanks for the feedback, anyway :)

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