Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Removing cursor: default; from icons #335

Merged
merged 4 commits into from
Feb 28, 2014
Merged

Removing cursor: default; from icons #335

merged 4 commits into from
Feb 28, 2014

Conversation

connors
Copy link
Collaborator

@connors connors commented Feb 27, 2014

Fixes #322

Not sure what the original thinking was for this but it was wack.

cc: @XhmikosR

@connors connors added this to the 2.0.1 milestone Feb 27, 2014
@XhmikosR
Copy link
Member

I'm afk; I can look closer tomorrow.

But are you sure about removing cursor pointer from .docs-jump-menu?

@connors
Copy link
Collaborator Author

connors commented Feb 27, 2014

Whoops, yeah you're right. I meant to remove that from .docs-nav-trigger instead.

@cvrebert cvrebert added the css label Feb 28, 2014
connors added a commit that referenced this pull request Feb 28, 2014
Removing `cursor: default;` from icons
@connors connors merged commit bbda606 into master Feb 28, 2014
@connors connors deleted the remove-cursor branch February 28, 2014 06:40
@mdo mdo mentioned this pull request Feb 28, 2014
@XhmikosR
Copy link
Member

Great job @connors !

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cursor issue
3 participants