Skip to content

Set 'display:inline-block' to the maximum handle. #201

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

Closed
wants to merge 1 commit into from
Closed

Set 'display:inline-block' to the maximum handle. #201

wants to merge 1 commit into from

Conversation

samechikson
Copy link

Previously, the right handle was being set 'display:null' for a range slider, so set it 'display:inline-block' to make it appear.

…handle of the range slider. Instead of setting 'display:null', do 'display:inline-block'
@ValentinH
Copy link
Member

Setting display to null is to remove any display value that has been set directly on the element. Thus, the pointer gets its default value which is indeed inline-block.

Why did you change that, do you have any problem to use range sliders?

@samechikson
Copy link
Author

I had to change it because in every instance of the range slider, the pointer was getting hidden. This was the only way to make the pointer show up.

@ValentinH
Copy link
Member

Can you post an example with JSFiddle please? What version are you using?

@ValentinH
Copy link
Member

I just received a PR fixing the same issue and I prefer how it's done there: #204 . The problem is due to JQuery I think.

@ValentinH ValentinH closed this Dec 17, 2015
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.

2 participants