[wp-trac] [WordPress Trac] #37004: Html5 / W3C Validation

[WordPress Trac]

#37004: Html5 / W3C Validation
-------------------------------------------------+-------------------------
 Reporter:  arena                                |       Owner:  mihai2u
     Type:  defect (bug)                         |      Status:  assigned
 Priority:  low                                  |   Milestone:  4.8
Component:  Administration                       |     Version:
 Severity:  trivial                              |  Resolution:
 Keywords:  good-first-bug has-screenshots has-  |     Focuses:
  patch                                          |  administration
-------------------------------------------------+-------------------------

Comment (by mihai2u):

 @afercia

 I've refreshed the patch and added back the colspan attributes even if
 they break validation.
 I pondered with the idea of using a data-colspan-diff="" attribute and use
 that in the JavaScript when present, but I discarded it as I don't think
 it brings much value. If you find value in going that approach to achieve
 validation on those particular areas (it's 3 templates receiving the HTML
 error), I can act on that.

 This is the error received: "Table columns in range 2…5 established by
 element td have no cells beginning in them." -> and that happens because
 the colspan is part of a template that's used by JS in order to be placed
 in the proper table which has the columns in that range. Statically
 speaking, if there's no JS making use of them, they indeed appear as
 inside a table with not enough columns to make the colspan valid.

 I hope all of the remaining code improvements can find their way onto
 4.8.1

 Thank you for your review.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/37004#comment:27>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform