step attribute doesn't trigger validity classes

When you are using step="1" a float value enables validity.stepMismatch. This is not reflected by the ng classes on the input.
e.g.

<input ng-model="example" type="number" step="1" />

steps to reproduce:

enter a float value
check validity.stepMismatch of the input (true)
check lack of ng-invalid class on the input

Affected version: 1.3.5

@Mouvedia

@Mouvedia it is next to impossible to assist you based on the info provided (or rather - lack of info...). Please provide a minimal, runnable reproduce scenario using http://plnkr.co/ or similar as per contribution guidelines

Ill update with more info. But it's pretty descriptive already.

But it's pretty descriptive already.

Yeh, might be for someone that sits in front of the code that exhibits the issue, but is far from being obvious for people not having exactly the same context as you have. For example, how can we guess what is the AngularJS version you are using?

Anyway, provide a live reproduce scenario using plunker and you will have a response in no time

@pkozlowski-opensource

@pkozlowski-opensource Seems no one will touch this for months now that you have put it in the "Purgatory" so Ill just use a custom validator to compensate for this obvious omission.

@Mouvedia

@Mouvedia we put things in the purgatory milestone by default where there is no enough info to confirm / reproduce the issue. It happens quite often, unfortunately, that people don't provide adequate info and don't come back to us after we request more info - we need a way of tracking those issues.

Now, if you want to speed up this issue resolution you can do one of those (which will be needed to fix the issue anyway):

send a pull request with a fix
check if the issue is still present in the latest version of AngularJS (once again, a plunker would avoid those back-and-forth)

You are welcomed to do any work-around on your side but if you want to see things improving on the core it would be great if you could send us a PR with a fix if you've investigated the issue already and qualified it as omission.

I can't imagine how much more strict you must be concerning PRs. So Ill just post the fix that Ill come up with when I am done.

Pawel, number and range inputs use the step attribute to rewrite inputs to be multiples of a given step value. We don't have handled for this for the number our range types, and when native validators are used, we ignore the step mismatch flags. That is the bug they're talking about

Should work on date month week datetime datetime-local time range number

feat(input[number]): support step

Navigation Menu

Repository navigation

step attribute doesn't trigger validity classes #10597

7 remaining items

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

step attribute doesn't trigger validity classes #10597

Description

Activity

pkozlowski-opensource commented on Dec 29, 2014

Mouvedia commented on Dec 29, 2014

pkozlowski-opensource commented on Dec 29, 2014

pkozlowski-opensource commented on Dec 29, 2014

Mouvedia commented on Dec 29, 2014

pkozlowski-opensource commented on Dec 29, 2014

Mouvedia commented on Dec 29, 2014

caitp commented on Dec 29, 2014

7 remaining items

jimmywarting commented on Sep 6, 2015

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions