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

Input required form field indactor missing #8114

Closed
ghaddon opened this issue Oct 29, 2017 · 13 comments
Closed

Input required form field indactor missing #8114

ghaddon opened this issue Oct 29, 2017 · 13 comments

Comments

@ghaddon
Copy link

ghaddon commented Oct 29, 2017

Bug, feature request, or proposal:

On Input control flagged as required does not show when using FormBuilder.

What is the expected behavior?

Would expect same behavior using template design method.

What is the current behavior?

No required indicator showing to the right of placeholder value when using the model method and Buildform.

What are the steps to reproduce?

Create input form control using FormBuilder and require field.

What is the use-case or motivation for changing an existing behavior?

Shows required indicator in template form creation but not FormBuilder method.

Which versions of Angular, Material, OS, TypeScript, browsers are affected?

I see it in Chrome Version: 61.0.3163.100 (Official Build) (64-bit).
Using Angular Version: 4.4.5
Using Angular Material Version: 2.0.0-beta.12
Using TypeScript Version: 2.3.3

Is there anything else we should know?

Nope

@donroyco
Copy link
Contributor

Can you add a Plunker or StackBlitz that describes your issue?

Plunker starter (using on @master): https://goo.gl/DlHd6U
StackBlitz starter (using latest npm release): https://goo.gl/wwnhMV

@willshowell
Copy link
Contributor

See #2574

@josephperrott
Copy link
Member

Closing as duplicate of #2574

@ghaddon
Copy link
Author

ghaddon commented Oct 31, 2017

Have not had time to create Plunker to show the problem. I read #2574 but, not sure it is the same. Can you direct me to the build to expect @2574 is fixed?

@willshowell
Copy link
Contributor

@ghaddon if you can show a reproduction that is distinctive from the description in #2574, then that would be worth looking into. But your description of the issue is nearly identical to that one.

@ghaddon
Copy link
Author

ghaddon commented Oct 31, 2017

Will,

Just completed sample of error I'm getting. can find onhttps://angular-material2-issue.stackblitz.io.

Thanks,
Gary

@willshowell
Copy link
Contributor

@ghaddon try forking the stackblitz. Your link just points to the default template.

@ghaddon
Copy link
Author

ghaddon commented Oct 31, 2017 via email

@willshowell
Copy link
Contributor

@ghaddon it's still empty

@ghaddon
Copy link
Author

ghaddon commented Oct 31, 2017 via email

@willshowell
Copy link
Contributor

@ghaddon yeah the issue you are describing is exactly the same as #2574

"MatInput doesn't set required asterisk when the validator is set in reactive forms"

@ghaddon
Copy link
Author

ghaddon commented Nov 1, 2017 via email

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants