Instead of duplicating a lot of code with xmlsitemap.xsl.js, we should convert it to use jQuery code. It will be easier to maintain as well.

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Can possibly even re-use the tablesort.js code as well.

EDIT: There is no jQuery tablesort JS included with core, so this would require http://tablesorter.com/docs/

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Assigned: Unassigned » dave reid
Status: Active » Fixed

Success! Converted to using jQuery and the tablesort plugin. Brings our custom XSL JS code from 149 lines to 15. Much, much more maintainable and integrated with Drupal. Committed to both 7.x-2.x and 6.x-2.x branches:
http://drupal.org/cvs?commit=305606
http://drupal.org/cvs?commit=305608

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Huge thank you to Christian for writing http://tablesorter.com/docs/. It truly rocks this situation for XML sitemap.

Status: Fixed » Closed (fixed)
Issue tags: -jQuery, -JavaScript, -XSL

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

Issue tags: +jQuery, +JavaScript, +XSL

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.