Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Disabled fieldset won't apply disabled styles to child elements #895

Closed
gustavohenke opened this issue Dec 10, 2014 · 4 comments
Closed

Disabled fieldset won't apply disabled styles to child elements #895

gustavohenke opened this issue Dec 10, 2014 · 4 comments
Assignees
@marcysutton
Milestone
0.8.3

Comments

@gustavohenke
Copy link

If using a disabled fieldset, styles won't be applied in its child elements.

Current results:

Note that the button is raised.
I expected the button to look like the disabled one from the Angular Material demo.

I think inputs don't need any special styles when they're disabled, except for cursor: default (which already is applied).

I'm using Angular Material v0.6.1.
@marcysutton marcysutton self-assigned this Dec 11, 2014
@ThomasBurleson ThomasBurleson modified the milestone: Backlog Dec 11, 2014
@marcysutton
Copy link
Contributor

Hi @gustavohenke, can you verify whether this is still an issue?

@gustavohenke
Copy link
Author

It works 👍
Thanks!

@gustavohenke
Copy link
Author

I should note that neither buttons or text fields are inheriting the styles anymore...
Demo: http://plnkr.co/edit/brYs1III2uzzsQ84yfJ6?p=preview

@gustavohenke gustavohenke reopened this Feb 4, 2015
@marcysutton
Copy link
Contributor

@gustavohenke I thought they looked fine in 0.7.1 but I do see the <md-input-container> isn't picking up the CSS for disabled inputs. Buttons look fine.

@marcysutton marcysutton reopened this Feb 9, 2015
marcysutton pushed a commit that referenced this issue Feb 10, 2015
fix(mdInput): css cascades from disabled fieldset
f79c618 
Closes #895
@marcysutton marcysutton mentioned this issue Feb 10, 2015
Closed
@ajoslin ajoslin added the in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs label Feb 10, 2015
@marcysutton marcysutton added resolution: fixed and removed in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs labels Feb 10, 2015
@marcysutton marcysutton modified the milestones: 0.8.0-rc1, Backlog Feb 10, 2015
marcysutton pushed a commit that referenced this issue Feb 10, 2015
fix(mdInput): css cascades from disabled fieldset
10258d5 
Closes #895
@antoinebrault antoinebrault mentioned this issue Mar 18, 2016
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@marcysutton marcysutton

Labels
None yet
Projects
None yet
Milestone
0.8.3
Development

Successfully merging a pull request may close this issue.

fix(mdInput): CSS cascades from disabled fieldsets angular/material
4 participants
@ThomasBurleson @gustavohenke @marcysutton @ajoslin