Skip to content
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

Layout Template Markdown Demos #117

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

flexyford
Copy link

The Layout Markdown Demos show the parameters and results that ember-collection is passing and receiving to/from the layout templates. It has been extended to all layout types, so click the Show Custom Layout Template button on any Grid Demo to see interactions between ember-collection and the cell-layout template.

Scroll-Position Layout Demo

Why add layout template demo?

The default grids are useful, but you may need to roll your own custom template solution. In this case it is helpful to know how ember-collection is using the layout templates. It took a couple of days to reverse-engineer the fixed layout template to really understand what ember-collection was doing. I created the Layout Demo to help bridge the knowledge gap for those who are also building their own custom templates.

Thanks to @lukemelia for his EmberNYC custom layout demo which made me attempt my own custom layout in the first place.

Note: This PR also includes README Updates

@flexyford
Copy link
Author

flexyford commented May 27, 2016

I'm curious what improvements could be made to the demo. I realize there are both documentation and demo changes in this PR. I believe a handful of ember developers could benefit from these layout template demos today. Please let me know some possible steps going forward. Thanks.

@varoot
Copy link

varoot commented Aug 30, 2016

Is this component still being actively developed? It seems there are a lot of unmerged PRs and no activities for a few months. Should I use this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants