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

Infinite resizing of iframe #7

Closed
jmav opened this issue Jan 21, 2014 · 12 comments
Closed

Infinite resizing of iframe #7

jmav opened this issue Jan 21, 2014 · 12 comments
Labels

Comments

@jmav
Copy link
Contributor

jmav commented Jan 21, 2014

If body has style height: 100% iframe is increasing it's height by few pixels.

Solution:
Preventive add style height: auto to body tag.

@davidjbradshaw
Copy link
Owner

Happy to take a pull request if you have a moment to look at that.

@davidjbradshaw
Copy link
Owner

Fixed in v1.1

@jmav
Copy link
Contributor Author

jmav commented Jan 23, 2014

David hi,

I’m preparing larger update on my fork, I will notify you on update.

Jure

On 23. jan. 2014, at 19:39, David J. Bradshaw notifications@github.com wrote:

Closed #7.


Reply to this email directly or view it on GitHub.

@davidjbradshaw
Copy link
Owner

Ok cool I just published a new version that uses mutationobserver to detect
DOM changes in the iFrame.

What are you working on?

Dave.

On Thursday, January 23, 2014, Jure notifications@github.com wrote:

David hi,

I’m preparing larger update on my fork, I will notify you on update.

Jure

On 23. jan. 2014, at 19:39, David J. Bradshaw <notifications@github.com<javascript:_e({}, 'cvml', 'notifications@github.com');>>
wrote:

Closed #7.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-33164839
.

David J. Bradshaw )'( dave@bradshaw.net

@jmav
Copy link
Contributor Author

jmav commented Jan 23, 2014

Change: previously loading & initializing of “iframe-resizer” plugin was made inside body tag, now the plugin can be defined in head tag and will be automatically initialized on jQuery document.ready event
Change: public “trigger” method for resizing is now renamed to: and triggered trough global event on window object (compatibility if “iframe-resizer” is not loaded)
Update: example to new definitions
Add: documentation: browser compatibility
Add: parameter to trigger event
Add: new parameter autoWindowResize - Default: true, disables resizing on window change event, useful for manual setting iframe dimensions.

On 23. jan. 2014, at 22:08, David J. Bradshaw notifications@github.com wrote:

Ok cool I just published a new version that uses mutationobserver to detect
DOM changes in the iFrame.

What are you working on?

Dave.

On Thursday, January 23, 2014, Jure notifications@github.com wrote:

David hi,

I’m preparing larger update on my fork, I will notify you on update.

Jure

On 23. jan. 2014, at 19:39, David J. Bradshaw <notifications@github.com<javascript:_e({}, 'cvml', 'notifications@github.com');>>
wrote:

Closed #7.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-33164839
.

David J. Bradshaw )'( dave@bradshaw.net

Reply to this email directly or view it on GitHub.

@davidjbradshaw
Copy link
Owner

Sounds good, only thing I ask is the iFrame code has no dependancies, as
it's designed to be a guest on other peoples sites.

On Thursday, January 23, 2014, Jure notifications@github.com wrote:

Change: previously loading & initializing of “iframe-resizer” plugin was
made inside body tag, now the plugin can be defined in head tag and will be
automatically initialized on jQuery document.ready event
Change: public “trigger” method for resizing is now renamed to: and
triggered trough global event on window object (compatibility if
“iframe-resizer” is not loaded)
Update: example to new definitions
Add: documentation: browser compatibility
Add: parameter to trigger event
Add: new parameter autoWindowResize - Default: true, disables resizing on
window change event, useful for manual setting iframe dimensions.

On 23. jan. 2014, at 22:08, David J. Bradshaw <notifications@github.com<javascript:_e({}, 'cvml', 'notifications@github.com');>>
wrote:

Ok cool I just published a new version that uses mutationobserver to
detect
DOM changes in the iFrame.

What are you working on?

Dave.

On Thursday, January 23, 2014, Jure <notifications@github.com<javascript:_e({}, 'cvml', 'notifications@github.com');>>
wrote:

David hi,

I’m preparing larger update on my fork, I will notify you on update.

Jure

On 23. jan. 2014, at 19:39, David J. Bradshaw <
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');><javascript:_e({}, 'cvml', '
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');>');>>
wrote:

Closed #7.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub<
https://github.com/davidjbradshaw/iframe-resizer/issues/7#issuecomment-33164839>

.

David J. Bradshaw )'( dave@bradshaw.net <javascript:_e({}, 'cvml',
'dave@bradshaw.net');>

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-33169541
.

David J. Bradshaw )'( dave@bradshaw.net

@jmav
Copy link
Contributor Author

jmav commented Jan 23, 2014

Sure, I did have problem with that, just added init retry methods.

J
On 24. jan. 2014, at 00:40, David J. Bradshaw notifications@github.com wrote:

Sounds good, only thing I ask is the iFrame code has no dependancies, as
it's designed to be a guest on other peoples sites.

On Thursday, January 23, 2014, Jure notifications@github.com wrote:

Change: previously loading & initializing of “iframe-resizer” plugin was
made inside body tag, now the plugin can be defined in head tag and will be
automatically initialized on jQuery document.ready event
Change: public “trigger” method for resizing is now renamed to: and
triggered trough global event on window object (compatibility if
“iframe-resizer” is not loaded)
Update: example to new definitions
Add: documentation: browser compatibility
Add: parameter to trigger event
Add: new parameter autoWindowResize - Default: true, disables resizing on
window change event, useful for manual setting iframe dimensions.

On 23. jan. 2014, at 22:08, David J. Bradshaw <notifications@github.com<javascript:_e({}, 'cvml', 'notifications@github.com');>>
wrote:

Ok cool I just published a new version that uses mutationobserver to
detect
DOM changes in the iFrame.

What are you working on?

Dave.

On Thursday, January 23, 2014, Jure <notifications@github.com<javascript:_e({}, 'cvml', 'notifications@github.com');>>
wrote:

David hi,

I’m preparing larger update on my fork, I will notify you on update.

Jure

On 23. jan. 2014, at 19:39, David J. Bradshaw <
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');><javascript:_e({}, 'cvml', '
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');>');>>
wrote:

Closed #7.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub<
https://github.com/davidjbradshaw/iframe-resizer/issues/7#issuecomment-33164839>

.

David J. Bradshaw )'( dave@bradshaw.net <javascript:_e({}, 'cvml',
'dave@bradshaw.net');>

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-33169541
.

David J. Bradshaw )'( dave@bradshaw.net

Reply to this email directly or view it on GitHub.

@jmav
Copy link
Contributor Author

jmav commented Jan 24, 2014

Please - merge it

https://github.com/jmav/iframe-resizer

On 24. jan. 2014, at 00:40, David J. Bradshaw notifications@github.com wrote:

Sounds good, only thing I ask is the iFrame code has no dependancies, as
it's designed to be a guest on other peoples sites.

On Thursday, January 23, 2014, Jure notifications@github.com wrote:

Change: previously loading & initializing of “iframe-resizer” plugin was
made inside body tag, now the plugin can be defined in head tag and will be
automatically initialized on jQuery document.ready event
Change: public “trigger” method for resizing is now renamed to: and
triggered trough global event on window object (compatibility if
“iframe-resizer” is not loaded)
Update: example to new definitions
Add: documentation: browser compatibility
Add: parameter to trigger event
Add: new parameter autoWindowResize - Default: true, disables resizing on
window change event, useful for manual setting iframe dimensions.

On 23. jan. 2014, at 22:08, David J. Bradshaw <notifications@github.com<javascript:_e({}, 'cvml', 'notifications@github.com');>>
wrote:

Ok cool I just published a new version that uses mutationobserver to
detect
DOM changes in the iFrame.

What are you working on?

Dave.

On Thursday, January 23, 2014, Jure <notifications@github.com<javascript:_e({}, 'cvml', 'notifications@github.com');>>
wrote:

David hi,

I’m preparing larger update on my fork, I will notify you on update.

Jure

On 23. jan. 2014, at 19:39, David J. Bradshaw <
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');><javascript:_e({}, 'cvml', '
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');>');>>
wrote:

Closed #7.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub<
https://github.com/davidjbradshaw/iframe-resizer/issues/7#issuecomment-33164839>

.

David J. Bradshaw )'( dave@bradshaw.net <javascript:_e({}, 'cvml',
'dave@bradshaw.net');>

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-33169541
.

David J. Bradshaw )'( dave@bradshaw.net

Reply to this email directly or view it on GitHub.

@davidjbradshaw
Copy link
Owner

Hi you will need to make a pull request on GitHub for me to be able to do a
merge.

Having had a quick look at the code, I'm not sure why the iframe needs the
settime out code added to it. The reason it was at the bottom, was that
this is current best pratise, not beacuse it needs to run last. The code is
triggered by an incoming postMessage, which can only happen after the page
has fully loaded. So it does not matter where on the page it is placed.

Also not clear why the retryPostInitMessagetoIframe() function is needed.
Currently the code ti trigger the iFrame is called twice, once when the
code runs, incase the iFrame is already loaded and once by an onload event
when the iFrame has completed loading. Between these two calls all
instances are covered. I'm guessing you added this because the timeout in
the iFrame created problems, as it delays that code until 10ms after the
iframe is ready and the unload event fires. But like I said you shouldn't
need that as it's the incoming messsage that triggers the code.

Happy to add the autoWindowResize flag. I think it needs to override the
interval timer and in version 1.1 the MutationObserver check as well.

Also your version of setHeightAuto is better than my one, as I forgot to
set auto height on the html tag.

So if you could make a github pull request with the autoWindowResize flag
and the setHeightAuto stuff in it, I would be happy to add those.

Also I'd rather leave the code at the bottom of the HTML, as this is best
pratise these days, as it makes the page load faster.

Cheers,
Dave.

On Fri, Jan 24, 2014 at 2:50 AM, Jure notifications@github.com wrote:

Please - merge it

https://github.com/jmav/iframe-resizer

On 24. jan. 2014, at 00:40, David J. Bradshaw notifications@github.com
wrote:

Sounds good, only thing I ask is the iFrame code has no dependancies, as
it's designed to be a guest on other peoples sites.

On Thursday, January 23, 2014, Jure notifications@github.com wrote:

Change: previously loading & initializing of “iframe-resizer” plugin
was
made inside body tag, now the plugin can be defined in head tag and
will be
automatically initialized on jQuery document.ready event
Change: public “trigger” method for resizing is now renamed to: and
triggered trough global event on window object (compatibility if
“iframe-resizer” is not loaded)
Update: example to new definitions
Add: documentation: browser compatibility
Add: parameter to trigger event
Add: new parameter autoWindowResize - Default: true, disables resizing
on
window change event, useful for manual setting iframe dimensions.

On 23. jan. 2014, at 22:08, David J. Bradshaw <
notifications@github.com<javascript:_e({}, 'cvml', '
notifications@github.com');>>
wrote:

Ok cool I just published a new version that uses mutationobserver to
detect
DOM changes in the iFrame.

What are you working on?

Dave.

On Thursday, January 23, 2014, Jure <notifications@github.com<javascript:_e({},
'cvml', 'notifications@github.com');>>
wrote:

David hi,

I’m preparing larger update on my fork, I will notify you on
update.

Jure

On 23. jan. 2014, at 19:39, David J. Bradshaw <
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');><javascript:_e({}, 'cvml', '
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');>');>>
wrote:

Closed #7.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub<

https://github.com/davidjbradshaw/iframe-resizer/issues/7#issuecomment-33164839>

.

David J. Bradshaw )'( dave@bradshaw.net <javascript:_e({}, 'cvml',
'dave@bradshaw.net');>

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub<
https://github.com/davidjbradshaw/iframe-resizer/issues/7#issuecomment-33169541>

.

David J. Bradshaw )'( dave@bradshaw.net

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-33190480
.

David J. Bradshaw )'( dave@bradshaw.net

@davidjbradshaw
Copy link
Owner

Hey,

just had a chance to read through the other stuff you did and want to say
thank you for all the documentation that I had been meaning to write and
also the changes to the trigger function, which has given me some other
ideas.

Can you send a pull request please.

D.

On Fri, Jan 24, 2014 at 12:23 PM, Dave Bradshaw
david.j.bradshaw@gmail.comwrote:

Hi you will need to make a pull request on GitHub for me to be able to do
a merge.

Having had a quick look at the code, I'm not sure why the iframe needs the
settime out code added to it. The reason it was at the bottom, was that
this is current best pratise, not beacuse it needs to run last. The code is
triggered by an incoming postMessage, which can only happen after the page
has fully loaded. So it does not matter where on the page it is placed.

Also not clear why the retryPostInitMessagetoIframe() function is needed.
Currently the code ti trigger the iFrame is called twice, once when the
code runs, incase the iFrame is already loaded and once by an onload event
when the iFrame has completed loading. Between these two calls all
instances are covered. I'm guessing you added this because the timeout in
the iFrame created problems, as it delays that code until 10ms after the
iframe is ready and the unload event fires. But like I said you shouldn't
need that as it's the incoming messsage that triggers the code.

Happy to add the autoWindowResize flag. I think it needs to override the
interval timer and in version 1.1 the MutationObserver check as well.

Also your version of setHeightAuto is better than my one, as I forgot to
set auto height on the html tag.

So if you could make a github pull request with the autoWindowResize flag
and the setHeightAuto stuff in it, I would be happy to add those.

Also I'd rather leave the code at the bottom of the HTML, as this is best
pratise these days, as it makes the page load faster.

Cheers,
Dave.

On Fri, Jan 24, 2014 at 2:50 AM, Jure notifications@github.com wrote:

Please - merge it

https://github.com/jmav/iframe-resizer

On 24. jan. 2014, at 00:40, David J. Bradshaw notifications@github.com
wrote:

Sounds good, only thing I ask is the iFrame code has no dependancies,
as
it's designed to be a guest on other peoples sites.

On Thursday, January 23, 2014, Jure notifications@github.com wrote:

Change: previously loading & initializing of “iframe-resizer” plugin
was
made inside body tag, now the plugin can be defined in head tag and
will be
automatically initialized on jQuery document.ready event
Change: public “trigger” method for resizing is now renamed to: and
triggered trough global event on window object (compatibility if
“iframe-resizer” is not loaded)
Update: example to new definitions
Add: documentation: browser compatibility
Add: parameter to trigger event
Add: new parameter autoWindowResize - Default: true, disables
resizing on
window change event, useful for manual setting iframe dimensions.

On 23. jan. 2014, at 22:08, David J. Bradshaw <
notifications@github.com<javascript:_e({}, 'cvml', '
notifications@github.com');>>
wrote:

Ok cool I just published a new version that uses mutationobserver
to
detect
DOM changes in the iFrame.

What are you working on?

Dave.

On Thursday, January 23, 2014, Jure <notifications@github.com<javascript:_e({},
'cvml', 'notifications@github.com');>>
wrote:

David hi,

I’m preparing larger update on my fork, I will notify you on
update.

Jure

On 23. jan. 2014, at 19:39, David J. Bradshaw <
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');><javascript:_e({}, 'cvml', '
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');>');>>
wrote:

Closed #7.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub<

https://github.com/davidjbradshaw/iframe-resizer/issues/7#issuecomment-33164839>

.

David J. Bradshaw )'( dave@bradshaw.net <javascript:_e({}, 'cvml',
'dave@bradshaw.net');>

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub<
https://github.com/davidjbradshaw/iframe-resizer/issues/7#issuecomment-33169541>

.

David J. Bradshaw )'( dave@bradshaw.net

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-33190480
.

David J. Bradshaw )'( dave@bradshaw.net

David J. Bradshaw )'( dave@bradshaw.net

@jmav
Copy link
Contributor Author

jmav commented Jan 27, 2014

Done

On 27. jan. 2014, at 11:09, David J. Bradshaw notifications@github.com wrote:

Hey,

just had a chance to read through the other stuff you did and want to say
thank you for all the documentation that I had been meaning to write and
also the changes to the trigger function, which has given me some other
ideas.

Can you send a pull request please.

D.

On Fri, Jan 24, 2014 at 12:23 PM, Dave Bradshaw
david.j.bradshaw@gmail.comwrote:

Hi you will need to make a pull request on GitHub for me to be able to do
a merge.

Having had a quick look at the code, I'm not sure why the iframe needs the
settime out code added to it. The reason it was at the bottom, was that
this is current best pratise, not beacuse it needs to run last. The code is
triggered by an incoming postMessage, which can only happen after the page
has fully loaded. So it does not matter where on the page it is placed.

Also not clear why the retryPostInitMessagetoIframe() function is needed.
Currently the code ti trigger the iFrame is called twice, once when the
code runs, incase the iFrame is already loaded and once by an onload event
when the iFrame has completed loading. Between these two calls all
instances are covered. I'm guessing you added this because the timeout in
the iFrame created problems, as it delays that code until 10ms after the
iframe is ready and the unload event fires. But like I said you shouldn't
need that as it's the incoming messsage that triggers the code.

Happy to add the autoWindowResize flag. I think it needs to override the
interval timer and in version 1.1 the MutationObserver check as well.

Also your version of setHeightAuto is better than my one, as I forgot to
set auto height on the html tag.

So if you could make a github pull request with the autoWindowResize flag
and the setHeightAuto stuff in it, I would be happy to add those.

Also I'd rather leave the code at the bottom of the HTML, as this is best
pratise these days, as it makes the page load faster.

Cheers,
Dave.

On Fri, Jan 24, 2014 at 2:50 AM, Jure notifications@github.com wrote:

Please - merge it

https://github.com/jmav/iframe-resizer

On 24. jan. 2014, at 00:40, David J. Bradshaw notifications@github.com
wrote:

Sounds good, only thing I ask is the iFrame code has no dependancies,
as
it's designed to be a guest on other peoples sites.

On Thursday, January 23, 2014, Jure notifications@github.com wrote:

Change: previously loading & initializing of “iframe-resizer” plugin
was
made inside body tag, now the plugin can be defined in head tag and
will be
automatically initialized on jQuery document.ready event
Change: public “trigger” method for resizing is now renamed to: and
triggered trough global event on window object (compatibility if
“iframe-resizer” is not loaded)
Update: example to new definitions
Add: documentation: browser compatibility
Add: parameter to trigger event
Add: new parameter autoWindowResize - Default: true, disables
resizing on
window change event, useful for manual setting iframe dimensions.

On 23. jan. 2014, at 22:08, David J. Bradshaw <
notifications@github.com<javascript:_e({}, 'cvml', '
notifications@github.com');>>
wrote:

Ok cool I just published a new version that uses mutationobserver
to
detect
DOM changes in the iFrame.

What are you working on?

Dave.

On Thursday, January 23, 2014, Jure <notifications@github.com<javascript:_e({},
'cvml', 'notifications@github.com');>>
wrote:

David hi,

I’m preparing larger update on my fork, I will notify you on
update.

Jure

On 23. jan. 2014, at 19:39, David J. Bradshaw <
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');><javascript:_e({}, 'cvml', '
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');>');>>
wrote:

Closed #7.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub<

https://github.com/davidjbradshaw/iframe-resizer/issues/7#issuecomment-33164839>

.

David J. Bradshaw )'( dave@bradshaw.net <javascript:_e({}, 'cvml',
'dave@bradshaw.net');>

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub<
https://github.com/davidjbradshaw/iframe-resizer/issues/7#issuecomment-33169541>

.

David J. Bradshaw )'( dave@bradshaw.net

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-33190480
.

David J. Bradshaw )'( dave@bradshaw.net

David J. Bradshaw )'( dave@bradshaw.net

Reply to this email directly or view it on GitHub.

@jmav
Copy link
Contributor Author

jmav commented Jan 27, 2014

Instead of initialization retry method I would use trigger from iframe, but problem is in unique identifier.

I would be neat solution.

J
On 27. jan. 2014, at 11:09, David J. Bradshaw notifications@github.com wrote:

Hey,

just had a chance to read through the other stuff you did and want to say
thank you for all the documentation that I had been meaning to write and
also the changes to the trigger function, which has given me some other
ideas.

Can you send a pull request please.

D.

On Fri, Jan 24, 2014 at 12:23 PM, Dave Bradshaw
david.j.bradshaw@gmail.comwrote:

Hi you will need to make a pull request on GitHub for me to be able to do
a merge.

Having had a quick look at the code, I'm not sure why the iframe needs the
settime out code added to it. The reason it was at the bottom, was that
this is current best pratise, not beacuse it needs to run last. The code is
triggered by an incoming postMessage, which can only happen after the page
has fully loaded. So it does not matter where on the page it is placed.

Also not clear why the retryPostInitMessagetoIframe() function is needed.
Currently the code ti trigger the iFrame is called twice, once when the
code runs, incase the iFrame is already loaded and once by an onload event
when the iFrame has completed loading. Between these two calls all
instances are covered. I'm guessing you added this because the timeout in
the iFrame created problems, as it delays that code until 10ms after the
iframe is ready and the unload event fires. But like I said you shouldn't
need that as it's the incoming messsage that triggers the code.

Happy to add the autoWindowResize flag. I think it needs to override the
interval timer and in version 1.1 the MutationObserver check as well.

Also your version of setHeightAuto is better than my one, as I forgot to
set auto height on the html tag.

So if you could make a github pull request with the autoWindowResize flag
and the setHeightAuto stuff in it, I would be happy to add those.

Also I'd rather leave the code at the bottom of the HTML, as this is best
pratise these days, as it makes the page load faster.

Cheers,
Dave.

On Fri, Jan 24, 2014 at 2:50 AM, Jure notifications@github.com wrote:

Please - merge it

https://github.com/jmav/iframe-resizer

On 24. jan. 2014, at 00:40, David J. Bradshaw notifications@github.com
wrote:

Sounds good, only thing I ask is the iFrame code has no dependancies,
as
it's designed to be a guest on other peoples sites.

On Thursday, January 23, 2014, Jure notifications@github.com wrote:

Change: previously loading & initializing of “iframe-resizer” plugin
was
made inside body tag, now the plugin can be defined in head tag and
will be
automatically initialized on jQuery document.ready event
Change: public “trigger” method for resizing is now renamed to: and
triggered trough global event on window object (compatibility if
“iframe-resizer” is not loaded)
Update: example to new definitions
Add: documentation: browser compatibility
Add: parameter to trigger event
Add: new parameter autoWindowResize - Default: true, disables
resizing on
window change event, useful for manual setting iframe dimensions.

On 23. jan. 2014, at 22:08, David J. Bradshaw <
notifications@github.com<javascript:_e({}, 'cvml', '
notifications@github.com');>>
wrote:

Ok cool I just published a new version that uses mutationobserver
to
detect
DOM changes in the iFrame.

What are you working on?

Dave.

On Thursday, January 23, 2014, Jure <notifications@github.com<javascript:_e({},
'cvml', 'notifications@github.com');>>
wrote:

David hi,

I’m preparing larger update on my fork, I will notify you on
update.

Jure

On 23. jan. 2014, at 19:39, David J. Bradshaw <
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');><javascript:_e({}, 'cvml', '
notifications@github.com <javascript:_e({}, 'cvml',
'notifications@github.com');>');>>
wrote:

Closed #7.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub<

https://github.com/davidjbradshaw/iframe-resizer/issues/7#issuecomment-33164839>

.

David J. Bradshaw )'( dave@bradshaw.net <javascript:_e({}, 'cvml',
'dave@bradshaw.net');>

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub<
https://github.com/davidjbradshaw/iframe-resizer/issues/7#issuecomment-33169541>

.

David J. Bradshaw )'( dave@bradshaw.net

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-33190480
.

David J. Bradshaw )'( dave@bradshaw.net

David J. Bradshaw )'( dave@bradshaw.net

Reply to this email directly or view it on GitHub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants