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

update(slider): improve disabled attribute validation #7310

Closed

Conversation

devversion
Copy link
Member

Improved the disabled attribute validation by only observing the attribute instead of using a getter function - as same as done in the autocomplete

@EladBezalel @topherfangio Please review

@devversion devversion added the needs: review This PR is waiting on review from the team label Feb 26, 2016
@EladBezalel
Copy link
Member

@devversion please check in #6538 before

@devversion
Copy link
Member Author

@EladBezalel Okay, can you remember about a similar issue to #6996? - if so let's close that as a dupe.

Yes, can see, you have really similar things done:

  • tabindex for disabled
  • demo updates
  • disabled attribute validation

But as I can see in your PR; you're still using the isDisabledGetter, and this one is not needed.

Update
As discussed with @EladBezalel he will take care about #6996 and this PR will only include the validation update, instead of the duplicated fix for #6996, which is already done #6538

@devversion devversion changed the title fix(slider): disable focus when disabled and improved disabled attribute validation update(slider): improve disabled attribute validation Feb 26, 2016
Improved the `disabled` attribute validation by only observing the attribute instead of using a getter function - as same as done in the `autocomplete`
@ThomasBurleson ThomasBurleson 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 Feb 29, 2016
@ThomasBurleson ThomasBurleson added this to the 1.0.7 milestone Feb 29, 2016
@ThomasBurleson ThomasBurleson self-assigned this Feb 29, 2016
ThomasBurleson pushed a commit that referenced this pull request Mar 1, 2016
Improved the `disabled` attribute validation by only observing the attribute instead of using a getter function - as same as done in the `autocomplete`

  Closes #7310
@devversion devversion deleted the fix/slider-disabled-focus branch April 19, 2016 19:52
aider pushed a commit to aider/material that referenced this pull request Jun 22, 2016
Improved the `disabled` attribute validation by only observing the attribute instead of using a getter function - as same as done in the `autocomplete`

  Closes angular#7310
aider pushed a commit to aider/material that referenced this pull request Sep 27, 2016
Improved the `disabled` attribute validation by only observing the attribute instead of using a getter function - as same as done in the `autocomplete`

  Closes angular#7310
aider pushed a commit to aider/material that referenced this pull request Dec 7, 2016
Improved the `disabled` attribute validation by only observing the attribute instead of using a getter function - as same as done in the `autocomplete`

  Closes angular#7310
aider pushed a commit to aider/material that referenced this pull request Feb 27, 2017
Improved the `disabled` attribute validation by only observing the attribute instead of using a getter function - as same as done in the `autocomplete`

  Closes angular#7310
aider pushed a commit to aider/material that referenced this pull request Sep 29, 2017
Improved the `disabled` attribute validation by only observing the attribute instead of using a getter function - as same as done in the `autocomplete`

  Closes angular#7310
aider pushed a commit to aider/material that referenced this pull request Apr 27, 2018
Improved the `disabled` attribute validation by only observing the attribute instead of using a getter function - as same as done in the `autocomplete`

  Closes angular#7310
aider pushed a commit to aider/material that referenced this pull request Aug 16, 2018
Improved the `disabled` attribute validation by only observing the attribute instead of using a getter function - as same as done in the `autocomplete`

  Closes angular#7310
aider pushed a commit to aider/material that referenced this pull request Mar 26, 2019
Improved the `disabled` attribute validation by only observing the attribute instead of using a getter function - as same as done in the `autocomplete`

  Closes angular#7310
aider pushed a commit to aider/material that referenced this pull request Apr 3, 2019
Improved the `disabled` attribute validation by only observing the attribute instead of using a getter function - as same as done in the `autocomplete`

  Closes angular#7310
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.

3 participants