-
Notifications
You must be signed in to change notification settings - Fork 34
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
Clear fast boot rendered head on browser boot #2
Conversation
Split initializers into fastboot & browser flavors. Use placeholder meta tags for marking extent of head.hbs content.
.nextUntil('meta[name="ember-cli-head-end"] ~') | ||
.addBack() | ||
.remove(); | ||
const container = instance.lookup ? instance : instance.container; |
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.
Should be able to remove this conditional, fastboot only supports ember 2.3.0+ so instance.lookup should always be present.
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.
Good point. 👍
This requires a shared package to ensure that fastboot version doesn't run when it ember-cli-fastboot is not present (since this addon still supports that world). @tomdale this is he scenario that I was explaining on the fastboot/browser initializer stripping PR. |
See ember-fastboot/ember-cli-fastboot#110 (comment) for details on filtering the initializers. |
Is this good to go? |
@tomdale haven't tested out the filter module as much as I plan to yet, but I should be able to cut a new release of this addon today. |
Clear fast boot rendered head on browser boot
@tomdale just published as 0.0.4 |
You're the hero Gotham needs. |
Split initializers into fastboot & browser flavors.
Use placeholder meta tags for marking extent of head.hbs content.