Skip to content

Specify cursor: pointer for slide.#480

Merged
XhmikosR merged 1 commit intomasterfrom
slider-cursor
Mar 12, 2014
Merged

Specify cursor: pointer for slide.#480
XhmikosR merged 1 commit intomasterfrom
slider-cursor

Conversation

@XhmikosR
Copy link
Copy Markdown
Member

@XhmikosR XhmikosR commented Mar 11, 2014

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
Copy Markdown
Collaborator

connors commented Mar 12, 2014

👍 I'm good with this for the docs.

@XhmikosR
Copy link
Copy Markdown
Member Author

XhmikosR commented Mar 12, 2014

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

@connors
Copy link
Copy Markdown
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
Copy Markdown
Member Author

XhmikosR commented Mar 12, 2014

OK, done.

XhmikosR added a commit that referenced this pull request Mar 12, 2014
@XhmikosR
Merge pull request #480 from twbs/slider-cursor
a50d492
@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
Closed
@alexfigueiredo
Copy link
Copy Markdown

alexfigueiredo commented Apr 23, 2014

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
Copy Markdown
Member Author

XhmikosR commented Apr 24, 2014

@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

Reviewers

No reviews

Assignees

@XhmikosR XhmikosR

Labels

css docs

Projects

None yet

Milestone

2.0.2

Development

Successfully merging this pull request may close these issues.

3 participants

@XhmikosR @connors @alexfigueiredo