[
https://issues.apache.org/jira/browse/SHINDIG-1767?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13273645#comment-13273645
]
[email protected] commented on SHINDIG-1767:
--------------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5067/#review7814
-----------------------------------------------------------
Ship it!
Committed r1337390
- Dan
On 2012-05-11 18:26:05, Dan Dumont wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/5067/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2012-05-11 18:26:05)
bq.
bq.
bq. Review request for shindig.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. With the latest changes from
https://issues.apache.org/jira/browse/SHINDIG-1766, the auto adjust of
dynamic-height and dynamic-width features was broken.
bq.
bq. The webkit height calculation is now the most reliable means of obtaining
the content height/width of the gadget in every browser I've tested, so I
cleaned that up for either height or width and moved it to the common feature.
bq.
bq.
bq. This addresses bugs SHINDIG-1745 and SHINDIG-1767.
bq. https://issues.apache.org/jira/browse/SHINDIG-1745
bq. https://issues.apache.org/jira/browse/SHINDIG-1767
bq.
bq.
bq. Diffs
bq. -----
bq.
bq.
http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/DynamicSizeDemo.xml
1333997
bq.
http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/DynamicSizeDemoHTML5.xml
PRE-CREATION
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/dynamic-height.height/dynamic-height-height.js
1333997
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/dynamic-size.util/dynamic-size-util.js
1333997
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/dynamic-size.util/feature.xml
1333997
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/dynamic-width.width/dynamic-width-width.js
1333997
bq.
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/dynamic-width/dynamic-width.js
1333997
bq.
bq. Diff: https://reviews.apache.org/r/5067/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. Tested in IE 7,8,9 Chrome, FF using quirks and html5
bq.
bq.
bq. Thanks,
bq.
bq. Dan
bq.
bq.
> Gadgets using gadgets.window.adjustHeight are no longer adjusting to the
> correct height
> ---------------------------------------------------------------------------------------
>
> Key: SHINDIG-1767
> URL: https://issues.apache.org/jira/browse/SHINDIG-1767
> Project: Shindig
> Issue Type: Bug
> Affects Versions: 2.5.0
> Reporter: Ryan Baxter
> Assignee: Dan Dumont
> Attachments: shindig.patch
>
>
> I am noticing that gadgets calling gadgets.window.adjustHeight are no longer
> agdjusting their height in the common container. For example try the ToDo
> gadget in the common container and you will notice the iframe will not grow
> to fit the whole gadget, however there will be a scroll bar allowing you to
> scroll.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
