Make WordPress Core

Changes between Version 2 and Version 3 of Ticket #33121, comment 16


Ignore:
Timestamp:
10/11/2018 04:10:17 PM (6 years ago)
Author:
azaozz
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #33121, comment 16

    v2 v3  
    33Looking at `data--invaild="gone"` and `data-also-invaild-="gone"`, it seems having two hyphens or a hyphen as last char of the data-* attribute name is valid per https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/data-* and https://www.w3.org/TR/REC-xml/#NT-Name. Also seems quite a few chars are valid there, but still thinking we should only support `a-z0-9_-`.
    44
    5 Also `preg_match( '/^' . preg_quote( $prefix ) . '(-[a-z0-9_]+)*$/', $name_low )` would mean we don't allow attribute manes containing two hyphens like `data-wp-id` (which is somewhat common).
     5Also `preg_match( '/^' . preg_quote( $prefix ) . '(-[a-z0-9_]+)*$/', $name_low )` would mean we don't allow attribute names containing two hyphens like `data-wp-id` (which is somewhat common).