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

Made it work with queries returning multiple elements, also it's possible to run box fit for the same element again (so you can update the text) #6

Merged
merged 5 commits into from
Aug 23, 2014

Conversation

pawelniewie
Copy link
Contributor

No description provided.

@ciaranj
Copy link

ciaranj commented Jan 13, 2014

+1 this PR fixes the issues I was seeing, this library does exactly what I wanted now :) Thank you (both)!

@JSteunou
Copy link
Contributor

+1

I found the same issue, and fix the same. Running on multiple elements is a plus.
Any chance you will consider this PR @michikono ?

@joshuapaling
Copy link

+1 I actually came here to fork this plugin and add this functionality myself - luckily I noticed it's already been added here!

michikono added a commit that referenced this pull request Aug 23, 2014
Made it work with queries returning multiple elements, also it's possible to run box fit for the same element again (so you can update the text)
@michikono michikono merged commit 3f25885 into michikono:master Aug 23, 2014
@michikono
Copy link
Owner

This has been accepted and I have subsequently significantly updated the documentation and demo pages.

@JSteunou
Copy link
Contributor

Seen that, great work! Good to see you dont let this plugin die, thank you @michikono!

@pawelniewie
Copy link
Contributor Author

👍

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.

5 participants