Code changed in jenkins
User: anafke
Path:
changelog.html
war/src/main/webapp/scripts/sortable.js
http://jenkins-ci.org/commit/jenkins/6566a0c7f9418469281f68cf243ca15d0ec32e95
Log:
[FIXED JENKINS-21729] Fix IE8 NPE with respect to sortability

IE8 (and I assume other browsers that use a doctype pre-html5) interpret the span outside of the link, as in HTML4.01 you needed the closing tag.
http://www.w3.org/TR/REC-html40/struct/global.html#h-7.5.4

The code was causing a NPE in IE8 and users could not sort the columns.

>> cell.innerHTML = '<a href="" class="sortheader">'this.getInnerText(cell)'<span class="sortarrow" /></a>';

>> cell.innerHTML ;
"<A class=sortheader href=""> S<SPAN class=sortarrow></A></SPAN>"

>> cell.innerHTML = '<a href="" class="sortheader">'this.getInnerText(cell)'<span class="sortarrow"></span></a>'

>> cell.innerHTML ;
"<A class=sortheader href=""> S<SPAN class=sortarrow></SPAN></A>"

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators.
For more information on JIRA, see: http://www.atlassian.com/software/jira

--
You received this message because you are subscribed to the Google Groups "Jenkins Issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to