On Sun, Aug 3, 2008 at 10:52 PM, RobG <rg...@iinet.net.au> wrote:
>
> On Jul 31, 4:44 am, Peter Michaux <petermich...@gmail.com> wrote:
>> Everything except the drag and drop code is now in beta for version
>> 0.2.
>
> [...]
>
>> If you would like to be able to create tickets please send me a
>> private email (not to the group!) with your preferred username and
>> password.
>
> Not sure I want to create tickets, but would like to be able to
> comment.
Certainly welcome.
> For example, in FORK.dom.addClass there is:
>
> el.className = [el.className, className].join(' ');
>
> I presume that is in resopnse to IE's poor string concatentation
> performance with the += compound operator. I recently installed
> IE8b1, so I can't test IE 6 or 7 any more (well, not easily), but
> apparently MS have done quite a bit to fix IE's string concatenation
> issues.
[snip test results]
Thanks for the info. I just sat down to make this change and when I
looked it, the use of join was already gone from the v0.2 beta code. I
had changed to the following so that there was no need to feature test
for join
el.className = el.className + ' ' + className;
It is now
el.className += ' ' + className;
Peter