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

Rangefield: Range field added #265

Merged
merged 2 commits into from
Mar 31, 2013
Merged

Rangefield: Range field added #265

merged 2 commits into from
Mar 31, 2013

Conversation

venca163
Copy link
Contributor

New HTML5 range field enhancement, discussed in #242 .

rilwis added a commit that referenced this pull request Mar 31, 2013
Rangefield: Range field added
@rilwis rilwis merged commit e604b3a into wpmetabox:master Mar 31, 2013
@rilwis
Copy link
Member

rilwis commented Mar 31, 2013

This field is very similar to number field which exists. Please take a look at it.

@funkatron82
Copy link
Contributor

Range is just a graphical version of number and a HTML5 version of
slider

On Sun, Mar 31, 2013 at 11:56 AM, Tran Ngoc Tuan Anh <
notifications@github.com> wrote:

This field is very similar to number field which exists. Please take a
look at it.


Reply to this email directly or view it on GitHubhttps://github.com//pull/265#issuecomment-15694138
.

@venca163 venca163 deleted the rangefield branch March 31, 2013 19:12
@venca163
Copy link
Contributor Author

Thanks for merge. I know range field is quite similar to number, but there are also differences. As I understand it, when using range, exact value is not so important and this goes hand in hand in how user interact with this field - by slider. Talking about support, finally Firefox is going to support it since version 22 . Untill then, in Firefox, this field is acting like a regular text input and the lower/upper bound has to be checked manually.

This pull request was closed.
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.

3 participants