tc39 / proposal-relative-indexing-method

A TC39 proposal to add an .at() method to all the basic indexable classes (Array, String, TypedArray)
MIT License
352 stars 19 forks source link

Web compatibility issue with Yui2 #31

Open evilpie opened 4 years ago

evilpie commented 4 years ago

We just enabled String.prototype.item in Firefox Nightly and broke http://bugzilla.mozilla.org. We are going to backout this patch for that reason.

The problem seems to be in the obsolete Yui2 library: https://github.com/yui/yui2/blob/master/build/dom/dom.js#L789. It now assumes that strings are some kind of array/collections object.

Constellation commented 4 years ago

We are going to revert String#item as well. https://bugs.webkit.org/show_bug.cgi?id=217449

tabatkins commented 4 years ago

Ah, well that makes the decision of whether to include String#item() or not easy - we'll just remove it.

Jack-Works commented 4 years ago

🧐