BREAKING(get rid of inner component)

[kybishop]

Initial spike of removing the inner component and switching to ember-popper 0.7.0, which uses a new approach for interacting with the popper element.

• Get dummy app working
  • Make sure options can still be toggled (aka fix bug in ember-popper)
  • Fix weird style bug
  • Fix renderInPlace (should add tests to prevent regressions) fix(toggling renderInPlace) kybishop/ember-popper#58
• Get tests working
• Make sure animations work as before
• Fix issue where position is altered between arg changes (like arrow true/false)

[kybishop]

I'll squash these commits when it's finally time to merge.

No rush, but @rwwagner90 I'd love your thoughts on this before I push it through.

[RobbieTheWagner]

@kybishop can you give me a TLDR of why we made this switch? There is no inner component at all now? Just one component? I'm curious how it effects the issues I was having with outer and inner classes and showing and hiding.

[kybishop]

Hey @rwwagner90, here are the bullet points:

1. The inner component was able to be removed thanks to ergonomic improvements to ember-popper.
2. class is now passed directly to the inner div
3. {{#ember-attacher}} is now {{#attach-popover}}
4. The popper <div> (the outer div) is now hidden/shown, and receives the aria-hidden attribute.

We can't remove the inner <div> entirely because popper.js's positioning CSS conflicts with the animation CSS. To get around the issue, we apply animations to the inner div instead.

In regards to your previous issues with class, they'd disappear entirely with this PR. This effectively gets us to parity/compatibility with ember-tooltips.

[RobbieTheWagner]

@kybishop sounds good to me! I briefly looked at the code, and didn't see anything glaringly wrong, so fine with me to ship it 👍