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

Support innerClass option #77

Merged
merged 6 commits into from Oct 18, 2017
Merged

Support innerClass option #77

merged 6 commits into from Oct 18, 2017

Conversation

@rwwagner90
Copy link
Collaborator

rwwagner90 commented Oct 17, 2017

No description provided.

@@ -10,7 +10,7 @@
target=target as |emberPopper|}}
{{#ember-attacher-inner animation=animation
arrow=arrow
class="inner"
class=(concat 'inner' ' ' innerClass)

This comment has been minimized.

Copy link
@kybishop

kybishop Oct 17, 2017

Owner

can't this just be (concat 'inner ' innerClass)?

This comment has been minimized.

Copy link
@rwwagner90

rwwagner90 Oct 17, 2017

Author Collaborator

Yep! Changing now

@kybishop kybishop merged commit 3f57227 into master Oct 18, 2017
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@kybishop kybishop deleted the inner-class branch Oct 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.