dcorking / google-wave-resources

Automatically exported from code.google.com/p/google-wave-resources
0 stars 0 forks source link

Bug: Dynamic width does not work if gadget is loaded with an initial width. #862

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
I made a pair of gadgets for this test case that just let you set their width 
and height dynamically.
The gadgets are hosted by the bot Gadgitty, from these two waves:
1. https://wave.google.com/wave/#restored:wave:googlewave.com!w%2BQxFVWiXHK
2. https://wave.google.com/wave/#restored:wave:googlewave.com!w%2BPXsBDBNDC
The waves are read-only but you can copy them to adjust the gadgets.

First case:

1. Insert a gadget that uses dynamic width and specifies a default width in its 
ModulePrefs.
   i.e. http://gadgitty.appspot.com/export?waveId=googlewave.com!w%2BQxFVWiXHK&ext=.xml

2. Call gadgets.window.adjustWidth in the gadget.

3. The width of the gadget's container will not change, although the width of 
the iframe itself will be changed.

This is because two container divs enclosing the gadget iframe have a width 
style set to the original width. They are not updated by adjustWidth(). They 
are only set once when the gadget loads.

Second case:

1. Insert a gadget that does not have a default width in its ModulePrefs.
   i.e. http://gadgitty.appspot.com/export?waveId=googlewave.com!w%2BPXsBDBNDC&ext=.xml

2. Call gadgets.window.adjustWidth in the gadget.

3. The width of the gadget's iframe and div containers will update as they are 
supposed to.

A DOM inspecter will show that the iframe container divs do not have a width 
style, and you will see that dynamic width works properly.

4. Navigate away from the wave and then back to it.

Now dynamic width will no longer work, just as in first case, and the iframe 
container divs will have the width style again.

Original issue reported on code.google.com by celehner1 on 24 Jul 2010 at 7:18

GoogleCodeExporter commented 8 years ago

Original comment by pamela.fox on 26 Jul 2010 at 2:05