RegExp.escape shoud escape "-" about twinkle HOT 3 CLOSED

Will investigate this one. Perhaps we can switch over to the MediaWiki-provided function?
http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/resources/jquery/jquery.mwExtension.js?view=markup#l17

from twinkle.

BTW: I just noticed the $.escapeRE() doesn't escape the "/" and considering that the following code shows "true, true, true" I think there is no need to escape the bar:

var x = new RegExp('/foo/'), y = new RegExp('\/foo\/'), z = new RegExp('\\/foo\\/');
alert( x.test('/foo/') +', '+ y.test('/foo/')+', '+ z.test('/foo/') );

Do you think it is necessary? If not we can switch to the current MW implementation.

from twinkle.

I agree with you. I don't know if space_fix is ever used but I have left it there just in case. All done here for now.

from twinkle.