Commit d1b71f9
committed
HTML API: Introduce wp_split_class_names().
This patch introduces a new CSS helper module containing a new function,
`wp_split_class_names()`. This function wraps some code to rely on the
HTML API to take an HTML `class` attribute value and return a
`Generator` to iterate over the classes in that value.
Many existing functions perform ad-hoc parsing of CSS class names,
usually by splitting on a space character. However, there are issues
with this approach:
- There is no decoding of HTML character references, which is normative
inside HTML attributes.
- There is no handling of null bytes.
- Class names can be split by more than just the space character.
- There is no handling of duplicates, and while mostly benign, code
forgetting to account for duplicates can lead to defects.
The new function handles the nuances to let developers focus on reading
CSS class names, adding new class names, and removing class names. This
serves a middleground between legacy code interacting with CSS class
names in isolation and code processing full HTML documents.1 parent 1885bee commit d1b71f9
2 files changed
+59
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
268 | 268 | | |
269 | 269 | | |
270 | 270 | | |
| 271 | + | |
271 | 272 | | |
272 | 273 | | |
273 | 274 | | |
| |||
0 commit comments