Specify `cursor: pointer` for `slide`. #480

Merged
merged 1 commit into from Mar 12, 2014

Projects

None yet

3 participants

@XhmikosR
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 docs css labels Mar 11, 2014
@XhmikosR XhmikosR added this to the 2.0.2 milestone Mar 11, 2014
@XhmikosR XhmikosR self-assigned this Mar 11, 2014
@connors
Member
connors commented Mar 12, 2014

👍 I'm good with this for the docs.

@XhmikosR
Member

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

@connors
Member
connors commented Mar 12, 2014

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

@XhmikosR
Member

OK, done.

@XhmikosR XhmikosR merged commit a50d492 into master Mar 12, 2014
@XhmikosR XhmikosR deleted the slider-cursor branch Mar 12, 2014
@connors connors referenced this pull request Mar 12, 2014
Closed

v2.0.2 ship list #484

@alexfigueiredo

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

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

@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