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

fixes #30 - allow passing arbitrary content to modal-footer element #33

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

atsu85
Copy link

@atsu85 atsu85 commented Oct 18, 2015

note i didn't update dist folder (as it would conflict with other merge request: #32 )

@plwalters
Copy link
Owner

@atsu85 This change negates the need for the button selectors = can you remove the two of them also please?

@atsu85
Copy link
Author

atsu85 commented Oct 19, 2015

thanks @PWKad for drawing my attention to it - now it is fixed.

(I did the change in my own source 3 months ago and as I didn't use elements matching those selectors everything worked for me - should have thought about it when i created the commit/pull request to contribute it back)

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

Successfully merging this pull request may close these issues.

2 participants