Semantic-ui: [Form Validation] not working for number inputs

[dilpreetsio]

I'm using semantic ui form validation in one of my projects in ember. The validation does not work for number type input.

Code Snippet:

level: {
          identifier : 'level',
          optional   : true,
          rules      : [
            {
              type   : 'integer',
              prompt : this.l10n.t('Please enter a number')
            }
          ]
        }

Template

{{input type='number' value=sponsor.level name='level' min=0}}

Note: Validation works if I remove optional: true

Please guide. Thanks

[awgv]

Hi @geekyd, I’ve just tested it, it works. Can you please elaborate, what did you mean when you said that it didn’t work, what happened exactly? Any console errors?

[dilpreetsio]

There are no console errors.
I'm able to enter e as input and its getting validated as an integer. It should prompt an error if e is present in the input. It seems to work if we remove optional.

[awgv]

@geekyd Alright, unfortunately, we can’t investigate it further if we can’t reproduce it in an isolated test case without Ember. I might’ve missed something when tested it, so please fork this JSFiddle and see if you can reproduce it there.

[dilpreetsio]

@Banandrew The Validation works for

<div class="field">
  <label class="required">Level</label>
  {{input type='number' value=sponsor.level name='level'}}
</div>

However the validation does not work for the nested fields.

<div class="fields">
<div class="two wide field">
  <label class="required">Level</label>
  {{input type='number' value=sponsor.level name='level'}}
</div>
</div>

Is it a known issue?

[awgv]

@geekyd Seems to be working fine: https://jsfiddle.net/Banandrew/3fwy7na6/

[dilpreetsio]

@Banandrew Please check https://jsfiddle.net/geekyd/7kwehhk7
Try non integer inputs eg. e
Thanks

[awgv]

@geekyd It’s the same JSFiddle that I posted.

[dilpreetsio]

@Banandrew Sorry about that, please check https://jsfiddle.net/geekyd/a2ktp4ap/

[awgv]

@geekyd Thank you, confirming the issue. It seems that you either have to set optional to false, or use the text type instead of the number.

[stale[bot]]

There has been no activity in this thread for 90 days. While we care about every issue and we’d love to see this fixed, the core team’s time is limited so we have to focus our attention on the issues that are most pressing. Therefore, we will likely not be able to get to this one.

However, PRs for this issue will of course be accepted and welcome!

If there is no more activity in the next 90 days, this issue will be closed automatically for housekeeping. To prevent this, simply leave a reply here. Thanks!