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

Specify cursor: pointer for slide. #480

Merged
merged 1 commit into from
Mar 12, 2014
Merged

Specify cursor: pointer for slide. #480

merged 1 commit into from
Mar 12, 2014

Conversation

XhmikosR
Copy link
Member

We could try grab too but doesn't seem to work with IE <= 10:

.slider .slide {
  cursor: -webkit-grab;
  cursor: -moz-grab;
  cursor: grab;
}

@XhmikosR XhmikosR added this to the 2.0.2 milestone Mar 11, 2014
@XhmikosR XhmikosR self-assigned this Mar 11, 2014
@connors
Copy link
Collaborator

connors commented Mar 12, 2014

👍 I'm good with this for the docs.

@XhmikosR
Copy link
Member Author

Which of the two ways? As much as I prefer grab it's not standard yet IIRC.

@connors
Copy link
Collaborator

connors commented Mar 12, 2014

Whoops sorry. I'm okay with using grap. I think it's supported well enough for our purposes.

@XhmikosR
Copy link
Member Author

OK, done.

XhmikosR added a commit that referenced this pull request Mar 12, 2014
@XhmikosR XhmikosR merged commit a50d492 into master Mar 12, 2014
@XhmikosR XhmikosR deleted the slider-cursor branch March 12, 2014 18:07
@connors connors mentioned this pull request Mar 12, 2014
@alexfigueiredo
Copy link

I would be nice to set grabbing when .slider is :active:

.slider .slide:active {
  cursor: -webkit-grabbing;
  cursor: -moz-grabbing;
  cursor: grabbing;
}

@XhmikosR
Copy link
Member Author

@alexxfsilva: you are welcome to make a PR for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants