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

IE11 md-checkbox ng-model-options not working with blur #7086

Closed
maureenvdr opened this issue Feb 9, 2016 · 1 comment
Closed

IE11 md-checkbox ng-model-options not working with blur #7086

maureenvdr opened this issue Feb 9, 2016 · 1 comment
Assignees
@devversion
Labels
pr: merge ready This PR is ready for a caretaker to review
Milestone
1.0.7

Comments

@maureenvdr
Copy link

If I add ng-model-options="{updateOn:'blur'}" to a md-checkbox it is not updating the ng-model after i click on something else. It works fine in firefox and chrome, but not in IE11.
You can reproduce this easly using the codepen that can be reached at https://material.angularjs.org/latest/demo/checkbox the one in basic usage, just add ng-model-options="{updateOn:'blur'}" to the first checkbox.
@devversion
Copy link
Member

Yep, can reproduce. Here a little demo to compare with the native checkbox (which works 👍) and the <md-checkbox> http://codepen.io/DevVersion/pen/BjGGNy

  • As we can see in comparison to Chrome. Chrome is selecting the actual <md-checkbox> and IE11 is selecting the <div class="md-container">

devversion added a commit to devversion/material that referenced this issue Feb 9, 2016
@devversion
fix(checkbox): fix IE11 focus for checkbox
141ff25 
At the moment IE11 is always selecting the div.md-container, but that will cause issues with the ngModel (options).
So we need to block pointer events for the container, because we will manage all from actual `<md-checkbox>`

Fixes angular#7086
devversion added a commit to devversion/material that referenced this issue Feb 9, 2016
@devversion
fix(checkbox): fix IE11 focus for checkbox
e49b4c0 
At the moment IE11 is always selecting the div.md-container, but that will cause issues with the ngModel (options).
So we need to block pointer events for the container, because we will manage all from actual `<md-checkbox>`

Fixes angular#7086
devversion added a commit to devversion/material that referenced this issue Feb 9, 2016
@devversion
fix(checkbox): fix IE11 focus for checkbox
5cbd6bc 
At the moment IE11 is always selecting the div.md-container, but that will cause issues with the ngModel (options).
So we need to block pointer events for the container, because we will manage all from actual `<md-checkbox>`

Fixes angular#7086
devversion added a commit to devversion/material that referenced this issue Feb 9, 2016
@devversion
fix(checkbox): fix IE11 focus for checkbox
d395c5d 
At the moment IE11 is always selecting the div.md-container, but that will cause issues with the ngModel (options).
So we need to block pointer events for the container, because we will manage all from actual `<md-checkbox>`

Fixes angular#7086
@devversion devversion mentioned this issue Feb 9, 2016
Closed
@ThomasBurleson ThomasBurleson added this to the 1.0.7 milestone Feb 27, 2016
@devversion devversion added the pr: merge ready This PR is ready for a caretaker to review label Feb 27, 2016
ThomasBurleson pushed a commit that referenced this issue Mar 1, 2016
@devversion @ThomasBurleson
fix(checkbox): fix IE11 focus for checkbox
1f6cd8f 
At the moment IE11 is always selecting the div.md-container, but that will cause issues with the ngModel (options).
So we need to block pointer events for the container, because we will manage all from actual `<md-checkbox>`

Fixes #7086

  Closes #7088
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@devversion devversion

Labels
pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Milestone
1.0.7
Development

No branches or pull requests
3 participants
@ThomasBurleson @devversion @maureenvdr