-
Notifications
You must be signed in to change notification settings - Fork 639
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Element#insert to accept array as insertions option #160
Comments
Matt Haggard Tag changed from “needs:discussion, performance, section:dom, syntax” to “needs:discussion, performance, section:dom, syntax” |
Tobie Langel State changed from “new” to “enhancement” |
Radoslav Stankov If is possible, could I ask what is the in mind for Prototype 2.0 ? :)
p.s. I haven't made decent tests, but maybe something like this could be faster:
|
dennis Importance changed from “” to “”
Perhaps the iteration could be improved, but THAT'S IT! |
dennis And when array handing is added, it will be easy to handle variable length arguments ...
|
dennis Ah, only Firefox agrees with this solution. I did some testing on current Chrome and Internet Explorer. Also, I did not thoroughly debug this but my guess is it's in the accessing/slicing of function arguments. |
dennis It seems Killemov implemented this quite elegantly in killemov@16ecf74043fa29950e2e3e... and he has made a pull request. |
previous lighthouse ticket #752
by Radoslav Stankov
In a lot of cases I need to insert a large group of elements to an element, and was wondering if it will be useful to have something like this
Plus if Prototype have this behavior it could be optimized with usages of documentFragments for example (аlthough tests will need be needed for this).
If such enhancement is accepted I will be more than happy to create patch for it
The text was updated successfully, but these errors were encountered: