Skip to content

/ servo

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

Implement range input sanitization #21952

Closed
wants to merge 3 commits into from
Closed

Implement range input sanitization #21952

wants to merge 3 commits into from

Commits on Oct 15, 2018

  1. Add a way not to sanitize values in an input until its creation is done

    @Eijebong
    Eijebong committed Oct 15, 2018
    cb87025

  2. Implement range input sanitization 

    Fixes #19773
    @Eijebong
    Eijebong committed Oct 15, 2018
    e790002

  3. Fix a range input test 

    It was setting the value to " 123" without setting a maximum above.
Since the specs specifies 100 to be the default max, the value we'd get
after sanitization would be 100, not 123. This fixes it by setting the
value to something that is between the default min/max.
    @Eijebong
    Eijebong committed Oct 15, 2018
    Loading status checks…
    e3e4771
You can’t perform that action at this time.