-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
changed component action to CamelCase. filled in readme. updated demo to show code. #16
Conversation
``` | ||
{{#frost-checkbox | ||
id='errored' | ||
classNameBindings='error' |
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.
class
instead of binding
(for most cases)?
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.
Actually, do we need this at all in the docs?
A few changes/questions to address prior to approval |
@psbanka @cstolli @job13er @sandersky More eyes on the proposal for the major version of checkbox to be included in |
👍 |
@@ -7,7 +7,7 @@ | |||
name=name | |||
update=(action 'input') | |||
}} | |||
<label for={{inputId}} class='label'>{{yield}} | |||
<label for={{inputId}} class='label'>{{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.
This needs to be conditional http://emberjs.com/api/classes/Ember.Component.html#property_hasBlock
@rox163 I assume this PR can be closed as the checkbox was moved to |
major