-
Notifications
You must be signed in to change notification settings - Fork 12
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
fix: remove wrapping <div>
#52
fix: remove wrapping <div>
#52
Conversation
This disables the wrapper div that EmberComponent implicitly adds to the DOM
🤦♂ That one slipped right past me. Sorry. 😅 Thanks for providing a PR right away! I have some minor suggestions. |
@buschtoens should be ok now! Do you want me to squash the install commits? Or are you squashing everything when merging? |
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.
Looks great! I'll squash merge and push new releases in a few minutes.
Thanks again.
* chore: Install ember-decorators v6.0.0 * fix: Disable the EmberComponent's wrapper element This disables the wrapper div that EmberComponent implicitly adds to the DOM * Revert "chore: Install ember-decorators v6.0.0" This reverts commit 0c96f58. * chore: Install @ember-decorators/component * Move ember-decorators/component to dependencies * purge company specific registry links from the lock file
The switch from sparkles to EmberComponent caused the appearance of wrapper divs.