Review Request: Media gadgets does not work on IE8

2011-10-21 Thread Jason Chiang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2527/ --- Review request for shindig, Ryan Baxter and Dan Dumont. Summary --- This pa

Re: Review Request: Media gadgets does not work on IE8

2011-10-21 Thread Jason Chiang
> On 2011-10-21 17:00:43, Dan Dumont wrote: > > http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/media-openGadgets/MediaUIOpenGadgets.js, > > line 120 > > > > > > pixel dimensions instead of

Re: Review Request: Media gadgets does not work on IE8

2011-10-21 Thread Ryan Baxter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2527/#review2761 --- Ship it! Committed revision 1187546. There seems to be a bunch of du

Re: Review Request: Media gadgets does not work on IE8

2011-10-21 Thread Dan Dumont
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2527/#review2751 --- Ship it! LGTM - Dan On 2011-10-21 16:26:52, Jason Chiang wrote: >

Re: Review Request: Media gadgets does not work on IE8

2011-10-21 Thread Dan Dumont
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2527/#review2748 --- http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer