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

Update iframeResizer.contentWindow.js #103

Closed
wants to merge 2 commits into from

Conversation

Jiachg
Copy link

@Jiachg Jiachg commented Oct 6, 2014

add setAutoResize for parentIFrame
modify line :530 , when autoResize true, enable sendToParent else ignore
add line : 208 --214 : add setAutoResize , when autoResize true, force to reset iframe

add setAutoResize for parentIFrame 
modify line :530 , when autoResize true,  enable sendToParent  else ignore 
add line : 208 --214 : add setAutoResize , when autoResize true, force to reset iframe
@davidjbradshaw
Copy link
Owner

Just had a look at this and it stops manual resizing, when autoResize is set to false. You would need to switch of the auto trigger events, rather than block the sending of the message for this to work in all cases.

when set autoresize false , stop the size change detect
@davidjbradshaw
Copy link
Owner

Sorry for the delay had another look at this and it doesn't work if you start with the autoResize option set to false. Think this is a bit of a bigger change to make it fully work.

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.

None yet

2 participants