the button.js and button-new.js are using escape() to work with URIs

http://www.w3schools.com/jsref/jsref_escape.asp is noting on this:
The escape() and unescape() functions should not be used to encode or decode URIs. Use encodeURI() and decodeURI() functions instead!

Should we?