-
Notifications
You must be signed in to change notification settings - Fork 286
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
Upgraded to ember 1.12.0 #485
Conversation
We chatted a bit about this in slack, and I think we decided that the extraction of I think that @patsy-issa is going to change that back (and confirm the tests still pass). |
c changing back and adding comments to the files. |
836a737
to
cde12be
Compare
Looks good to me. @teddyzeenny - r? |
mouseEnter(e) { | ||
this.get('controller').send('previewLayer', this.get('node')); | ||
e.stopPropagation(); | ||
}, | ||
|
||
/** | ||
* @property mouseLeave |
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.
s/property/method
f9afd68
to
c436c9f
Compare
Can you remove "Moved required properties into list-view mixin" from the commit message? |
Upgraded ember list view to v 0.0.6 Added SafeString to style bindings Part of emberjs#360
c436c9f
to
524afb5
Compare
Updated commit message and added |
🍻 |
Upgraded ember list view to v 0.0.6
Added SafeString to style bindings
Part of #360