-
Notifications
You must be signed in to change notification settings - Fork 248
Web Component support: binding to properties #1277
Conversation
transformers: | ||
- angular | ||
- $dart2js: | ||
minify: false |
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.
Any particular reason for minify: false
?
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.
nope, removed.
Would you like to add an E2E test? 😄 |
{ | ||
"name": "paper-example", | ||
"version": "0.0.1", | ||
"homepage": "https://github.com/jbdeboer/angular.dart", |
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.
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.
done
@@ -87,7 +87,7 @@ class DirectiveSelector { | |||
if (attrName.startsWith("on-")) { | |||
builder.onEvents[attrName] = value; | |||
} else if (attrName.startsWith("bind-")) { | |||
builder.bindAttrs[attrName] = _astParser(value, formatters: _formatters); | |||
builder.bindAttrs[attrName.substring(5)] = _astParser(value, formatters: _formatters); |
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.
const _BIND_PREFIX_LENGTH = 5; // "bind-" ?
LGTM, with a few questions and comments |
|
||
<p> | ||
Change the value through an ng-model bound input box: | ||
<input type="text" ng-model="curValue"> |
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.
<label>Change the value through an ng-model bound input box: <input type="text" ng-model="curValue"></label>
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.
done
No description provided.