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

Validators for input type="radio" don't work. #82

Closed
GrzegorzStanczyk opened this issue Mar 12, 2021 · 5 comments
Closed

Validators for input type="radio" don't work. #82

GrzegorzStanczyk opened this issue Mar 12, 2021 · 5 comments

Comments

@GrzegorzStanczyk
Copy link

Control is valid when Validators.required is set.

I'm submitting a...


[X] Bug report

Current behavior

Radio control is always valid

Expected behavior

To work same as native FormControl

Minimal reproduction of the problem with instructions

https://stackblitz.com/edit/angular-ivy-emnm3t?file=src%2Fapp%2Fapp.component.ts
Switch FormControl between @angular/forms and @ngneat/reactive-forms and look at valid/invalid status.

What is the motivation / use case for changing the behavior?

Validate the radio control

Environment


Angular version: ^11.0.8

Browser:
- [x] Chrome (desktop) version XX
 
For Tooling issues:
- Node version: v14.16.0
- Platform:  Windows
@NetanelBasal
Copy link
Member

Screen Shot 2021-03-12 at 16 41 18

@kbrilla
Copy link

kbrilla commented Mar 12, 2021

Hi,
@NetanelBasal what version of Angular did You use and what browser? I tested it on the newest versions of Chrome, Edge, Chrome canary, and Firefox. I will check Angular 10 tomorrow, maybe that will be it.
Did it work all right for You on stackblitz provided?

@NetanelBasal
Copy link
Member

I tried it on our playground here.

@diego-anthony
Copy link

diego-anthony commented Mar 21, 2021

I have the same result in stackblitz, it is more; the same result for input text type, but I replicated it in a local project and it works fine.

Browser: Opera

@NetanelBasal
Copy link
Member

Closing this issue for now.

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

No branches or pull requests

4 participants