-
-
Notifications
You must be signed in to change notification settings - Fork 131
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
Add some cellType and columnType docs #91
Add some cellType and columnType docs #91
Conversation
💯 thank you! |
No problem @taras! Trying to help out where I can 😄 |
@rwwagner90 thanks for taking the time to document the cell and column types, I think it looks great 😺! In terms of styling, Im not really a fan of declaring |
@offirgolan I just try to destructure whatever I can. Is there a reason you particularly do not like doing so with Component? |
Honestly, its nothing more than of a personal preference / pet-peeve |
@offirgolan I'm just not sure what makes component special. I thought the standards were you should destructure everything out of the Ember namespace that you can rather than using |
Thanks @rwwagner90 |
I did some destructuring as well, for consistency of not using
Ember.
, but destructuring instead.