Using jQuery selectors etc would:
* make the code smaller, e.g. $('.edit-remove-text').val(decodeURIComponent('foo')); instead of document.getElementById('edit-remove-text').value = decodeURIComponent('foo');
* Make it easier to add additional dynamic functionality, e.g. client-side validation.

CommentFileSizeAuthor
#3 taxonomy_redirect-n392626_js.patch1.81 KBdamienmckenna
#2 taxonomy_redirect-n392626.patch1.56 KBdamienmckenna

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Correction, that should be:
$('#edit-remove-text').val(decodeURIComponent('foo'));

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new taxonomy_redirect-n392626.patch1.56 KB

Initial patch.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new taxonomy_redirect-n392626_js.patch1.81 KB

Here's a patch for the taxonomy_redirect.js file.

agileware’s picture

agileware commented
Status: Needs review » Fixed

Thanks for this patch.

I have applied this now. It should be in the dev version sometime today and then in the next release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.