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

Glimmer Components Checklist #12129

Closed
12 tasks
rwjblue opened this issue Aug 17, 2015 · 3 comments
Closed
12 tasks

Glimmer Components Checklist #12129

rwjblue opened this issue Aug 17, 2015 · 3 comments

Comments

@rwjblue
Copy link
Member

rwjblue commented Aug 17, 2015

Post merge pending items from #12011:

  • Fix top-level {{partial}} case (0604db1)
  • Make sure that people don't accidentally use old-style APIs for top-level elements in GlimmerComponent (tagName)
  • Refactor to separate conceptually different code paths that happen to share code, such as...
    • build-component-template
    • component node manager?
  • make recursive invocation work (<my-foo> component as the top-level element inside <my-bar>'s layout)
  • discuss a path for properties vs. attributes being plausibly aligned between normal elements and components
  • Make sure elementId is set on GlimmerComponent
  • Fix skipped tests.
  • Make GlimmerComponent isComponentFactory
  • Export GlimmerComponent as a module
  • Add flag to allow a component author to use new-semantics and provide a deprecation when the component is invoked with curlies (providing a bridge from {{foo-bar blah="baz"}} to <foo-bar blah="baz"></foo-bar>).
@rwjblue
Copy link
Member Author

rwjblue commented Aug 17, 2015

/cc @wycats @chancancode

@mixonic
Copy link
Member

mixonic commented Mar 27, 2016

@rwjblue can this be closed?

@rwjblue
Copy link
Member Author

rwjblue commented Mar 28, 2016

Ya. Will close for now, if/when we start pushing on GlimmerComponent again, we can reevaluate.

@rwjblue rwjblue closed this as completed Mar 28, 2016
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

No branches or pull requests

2 participants