feat(navBar): add disabled attribute #10202
Conversation
@@ -28,5 +32,8 @@ | |||
|
|||
<md-checkbox ng-model="disableInkBar">Disable Ink Bar</md-checkbox> | |||
|
|||
<md-checkbox ng-model="secondTabDisabled" aria-label="Disable item two?" style="margin: 5px;"> | |||
Disable Second Tab? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would look better to remove the ?
here (also in the aria-label
)
@@ -442,6 +444,7 @@ function MdNavItem($$rAF) { | |||
'mdNavSref': '@?', | |||
'srefOpts': '=?', | |||
'name': '@', | |||
'disabled': '@?' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this necessary, if we already use the attrs.$observe
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, you're right. It isn't necessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@maxjoehnk Can you rebase? |
@kara Done |
* Add disabled attribute to navitem * Update demo to showcase ng-disabled and disabled usage * Add tests * Ignore navbar specific button styling when disabled fixes #9667
@maxjoehnk - something bad happened with your rebase. We now have 40 files to merge. That will not be allowed. Please resubmit a new PR based on the lastest from https://github.com/angular/material |
fixes #9667