----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2576/#review2879 -----------------------------------------------------------
LGTM just one small nit http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/selection/selection.js <https://reviews.apache.org/r/2576/#comment6434> Ok now I am being picky, but could you move the .length calls down into the for loops? (and the ones below) Just one less line of code...trying to make Dan happy :) - Ryan On 2011-10-27 14:00:05, Jason Chiang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2576/ > ----------------------------------------------------------- > > (Updated 2011-10-27 14:00:05) > > > Review request for shindig, Ryan Baxter and Dan Dumont. > > > Summary > ------- > > IE8 doesn't support both "forEach" and "indexOf" Javascript methods, replace > these methods with for loop. > > > This addresses bug SHINDIG-1652. > https://issues.apache.org/jira/browse/SHINDIG-1652 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/selection/selection.js > 1189339 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/selection/selection_container.js > 1189339 > > Diff: https://reviews.apache.org/r/2576/diff > > > Testing > ------- > > Test it with the sample container in IE8 > > > Thanks, > > Jason > >
