Gotcha. If you fix it so that array.push works, it'll throw another error. I kept fixing them until I got to the 'for el in array' error, which I didn't know how to fix. So, best not to include prototype.js if you want to support IE 5, or if you know some magic that makes the for...in work, then go nuts.
Greg > -----Original Message----- > From: [EMAIL PROTECTED] [mailto:rails-spinoffs- > [EMAIL PROTECTED] On Behalf Of Maninder, Singh > Sent: Monday, June 12, 2006 10:10 AM > To: rails-spinoffs@lists.rubyonrails.org > Subject: RE: [Rails-spinoffs] Array.prototype.push() > > Thanks Greg. > > I know prototype doesn't work on IE5. > > Neither does my application, but for some reason my decider page (ajax vs > non-ajax) was including the prototype library and I got greedy to make > just that page work with the > call to prototype.js (even though nothing is called from that file). Was > getting an error on the line with push() so came up with that question. > > :) > -Mandy. > > > -----Original Message----- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] Behalf Of > Gregory Hill > Sent: Monday, June 12, 2006 8:16 PM > To: rails-spinoffs@lists.rubyonrails.org > Subject: RE: [Rails-spinoffs] Array.prototype.push() > > > IE 5 does not have it, but prototype does not work in IE 5 anyway. The > for...in structure causes IE 5 to throw JS errors just by including > prototype.js. > > IE 5.5 works fine and has array.push (I may be wrong, but I believe the > only changes from 5.5 to 6 are changes in CSS support. 5 to 5.5 was a > big change in all areas). > > So, there's no need for array.push in prototype because it doesn't add > any browsers to the support list. > > Greg > > > -----Original Message----- > > From: [EMAIL PROTECTED] > [mailto:rails-spinoffs- > > [EMAIL PROTECTED] On Behalf Of Maninder, Singh > > Sent: Sunday, June 11, 2006 11:44 PM > > To: rails-spinoffs@lists.rubyonrails.org > > Subject: RE: [Rails-spinoffs] Array.prototype.push() > > > > Yeah, it is, but IE 5 & I think 5.5 don't have it.... > > > > Prototype 1.3.1 (I think?) had it :) > > > > -----Original Message----- > > From: [EMAIL PROTECTED] > > [mailto:[EMAIL PROTECTED] Behalf Of Ed > C. > > Sent: Saturday, June 10, 2006 9:14 PM > > To: rails-spinoffs@lists.rubyonrails.org > > Subject: Re: [Rails-spinoffs] Array.prototype.push() > > > > > > Isn't the native Array object's push() method already supported by > > browsers? (As in, it doesn't need to be redefined by Prototype) > > > > On 6/10/06, Maninder, Singh <[EMAIL PROTECTED]> wrote: > > > Does anyone have any idea why Array.prototype.push() was removed > from > > prototype? > > > _______________________________________________ > > > Rails-spinoffs mailing list > > > Rails-spinoffs@lists.rubyonrails.org > > > http://lists.rubyonrails.org/mailman/listinfo/rails-spinoffs > > > > > _______________________________________________ > > Rails-spinoffs mailing list > > Rails-spinoffs@lists.rubyonrails.org > > http://lists.rubyonrails.org/mailman/listinfo/rails-spinoffs > _______________________________________________ > Rails-spinoffs mailing list > Rails-spinoffs@lists.rubyonrails.org > http://lists.rubyonrails.org/mailman/listinfo/rails-spinoffs > _______________________________________________ > Rails-spinoffs mailing list > Rails-spinoffs@lists.rubyonrails.org > http://lists.rubyonrails.org/mailman/listinfo/rails-spinoffs _______________________________________________ Rails-spinoffs mailing list Rails-spinoffs@lists.rubyonrails.org http://lists.rubyonrails.org/mailman/listinfo/rails-spinoffs
