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

Prevent invalid ratio calculation and fitting #36

Conversation

zarko-tg
Copy link
Contributor

No description provided.

@zarko-tg
Copy link
Contributor Author

Demo of the problem here: http://codepen.io/zarko/pen/avmGZV

@andreialecu
Copy link

The problem here is that when the text is in a container that is hidden with display:none, offsetHeight and offsetWidth will return 0, since it will be set invisible as well.

+1 for this fix. I also stumbled upon this bug. @patrickmarabeas a merge and new release with this would be great.

@MonkeyDo
Copy link

This solved a problem I had when using this with Ionic and cached pages.

Please do merge in !

@patrickmarabeas
Copy link
Owner

Sorry guys! I've been swamped - will get to releasing a new version with latest round of fixes ASAP!

patrickmarabeas added a commit that referenced this pull request Jan 17, 2016
…n_and_fitting

Prevent invalid ratio calculation and fitting
@patrickmarabeas patrickmarabeas merged commit 7de9c5b into patrickmarabeas:master Jan 17, 2016
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.

4 participants