----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/2576/#review2872 -----------------------------------------------------------
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/selection/selection.js <https://reviews.apache.org/r/2576/#comment6429> Jason we can simplify this code a little, instead of finding the index and then doing the splice you can just do the splice and break. Same thing with the code below too. - Ryan On 2011-10-26 20:10:27, Jason Chiang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/2576/ > ----------------------------------------------------------- > > (Updated 2011-10-26 20:10:27) > > > 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 > >
