-
Notifications
You must be signed in to change notification settings - Fork 27.4k
input[checkbox] doesn't catch checked in scope directive #13767
Comments
Seems to have been caused by 6f3b862 in combination with replace: true (if you remove replace, it works). Since replace is deprecated, I wouldn't hold out for a fix. If you want to provide a PR that fixes it, I'll be happy to take a look. |
Oh, i also works if you use a different name for the checked attribute: https://plnkr.co/edit/yHiTck5stWNkhASsWA7t?p=preview |
The problem is that |
This is the relevant commit: 8a1eb16 |
So since we always convert boolean attributes to a boolean value, you cannot use interpolation inside them as this will always be converted to |
Ah, I was looking for this previous commit. Note that the plnkr only fails once you go to 1.4.4, though. |
Well this commit was part of a bit of reverting and unreverting work at the time so it is not exactly clear where it appears |
Thanks a lot for your reactivity, the fix and globally for your work. I'll try that tomorrow. |
The attribute "checked" is not recovered in the scope of a directive.
This seems broken since version 1.4.4
https://plnkr.co/edit/0j7Y9mkeP4m2waCLYLxU?p=preview
Working on 1.3.15
https://plnkr.co/edit/GN4K5EdtfHxPpe7x3OCc?p=preview
The text was updated successfully, but these errors were encountered: