-
-
Notifications
You must be signed in to change notification settings - Fork 983
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
Endless loop if both JS files are added #58
Comments
Do your iFrames have IDs on them? |
They didn't. Now they have and I'm using |
Oh the HTML tag, not the JS call. |
You mean: <iframe id="inner" src="inner.html" width="100%" ></iframe> ? Yep, I added that in the last comment and additionally used the |
What happens in the endless loop? Which browser? Does the nested example work for you? |
Google Chrome v35, Firefox 24 and Firefox 29 under Linux. Sorry, hadn't seen the nested example. Same problem: it works well when opening the top frame and then clicking on the Nested link. But opening the nested frame directly, there is an endless loop where the height is being continuously increased. |
Oh now I understand the issue. Will take a look. |
Does this work for you now? http://davidjbradshaw.com/iframe-resizer/example/frame.nested.html?1 |
Both in Firefox and Chrome it does not enter in a loop anymore. In Chrome it works perfectly, but in Firefox the iframe is not properly resized (see screenshot). |
Bugger seems FireFox has a function stack scoping bug in it JS implementation. Can you give it another go. D. |
Thank you! It works like a charm in Chrome, Firefox 24 and 29. Thank you! |
Cool will publish the fix shortly as v2.5.1 |
I have three HTML files:
Each one contains the other. Both
inner.html
andsuperouter.html
work as expected, butouter.html
enters in an endless loop.I see that changing
msgId
, it works. In a similar scenario I have:Here, the three files work as expected. But I'd like to avoid changing the JS files in every update, if possible. Is there anything I'm missing? Is there any other way to make this scenario work?
Thank you,
The text was updated successfully, but these errors were encountered: