-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
MdSelect inside disabled fieldset should be disabled #4744
Comments
Related to #2763 |
Should be resolved by #4334. |
Seems to be tracked in angular/angular#13856. |
Will this be resolved by angular/angular#13856? The Material Select component is different than the other form control components because it does not render an actual |
I think all related issues have been funneled towards angular/angular#35309 -- do I understand correctly that resolving that will fix this as well? |
Closing this since the underlying issue is tracked on angular/angular#35309 |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Bug, feature request, or proposal:
Bug
What is the expected behavior?
A md-select element should be disabled if it is inside a disabled fieldset like other fields such as input, textarea, ...
What is the current behavior?
You can use the md-select component even if its parent fieldset is disabled.
What are the steps to reproduce?
http://plnkr.co/edit/bXjtAGhH2jpM4jrSB3Pg?p=preview
What is the use-case or motivation for changing an existing behavior?
When you wish to disable a full form it is very convenient to use a disabled fieldset. It will in fact disable everything and removes the need to disable each single field one by one.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular : "@angular/core": "4.1.3",
Material : "@angular/material": "2.0.0-beta.5",
Typescript : "typescript": "^2.3.3"
OS: OSX & Windows
Browsers : All
Is there anything else we should know?
None that I know of!
The text was updated successfully, but these errors were encountered: