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

fix(chips): hide remove button on custom template when readonly #4090

Closed
wants to merge 1 commit into from

Conversation

typotter
Copy link
Contributor

Add ng-if to the remove button, conditional on not readonly. Also tweak the custom input styling in the demo, and add the 'readonly' class to md-chip when it is readonly.

fixes #3697

@typotter typotter added the needs: review This PR is waiting on review from the team label Aug 11, 2015
Add ng-if to the remove button, conditional on not readonly. Also tweak the custom input styling in the demo, and add the 'readonly' class to md-chip when it is readonly.

fixes angular#3697
@typotter
Copy link
Contributor Author

Will need to merge this after PR#4114 goes in, but @jelbourn, care to take a look?

@jelbourn
Copy link
Member

LGTM

@typotter typotter added pr: merge ready This PR is ready for a caretaker to review and removed needs: review This PR is waiting on review from the team labels Aug 12, 2015
kennethcachia pushed a commit to kennethcachia/material that referenced this pull request Sep 23, 2015
Add ng-if to the remove button, conditional on not readonly. Also tweak the custom input styling in the demo, and add the 'readonly' class to md-chip when it is readonly.

fixes angular#3697. closes angular#4090.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Readonly chips allows deletion with custom template
3 participants