#58551 closed enhancement (fixed)
Add kses support for complex CSS grid values including `repeat`, `auto-fill` and `auto-fit`
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 6.3 | Priority: | normal |
| Severity: | normal | Version: | 6.3 |
| Component: | Formatting | Keywords: | has-patch has-unit-tests add-to-field-guide |
| Focuses: | Cc: |
Description
In order for the new grid layout type from Gutenberg https://github.com/WordPress/gutenberg/pull/49018 to work, kses needs to support a variety of grid values.
For the current implementation only repeat and auto-fill are strictly necessary, but this may change in the future so it might be worth considering support for auto-fit, max-, min- and fit-content. The grid spec is getting built out fast, and soon there will also be good browser support for subgrid, with masonry following behind. See https://developer.mozilla.org/en-US/docs/Web/CSS/grid-template-columns for a list of possible values.
Change History (7)
This ticket was mentioned in PR #4637 on WordPress/wordpress-develop by @isabel_brison.
2 years ago
#1
- Keywords has-patch has-unit-tests added
This ticket was mentioned in PR #4637 on WordPress/wordpress-develop by @isabel_brison.
2 years ago
#2
Trac ticket: https://core.trac.wordpress.org/ticket/58551
Adds support for the CSS repeat function.
The Trac ticket mentions support for auto-fit and -fill, but in testing I realised those are already implicitly supported so no changes needed there.
To test this change in the front end, apply the patch on top of #4625 and follow testing steps from that PR.
@isabel_brison commented on PR #4637:
2 years ago
#4
Not sure about the removal of the aria-* attributes (seems like a typo/accidental)?
Definitely accidental! I wonder how it happened, I didn't edit those lines manually 🤔
#5
@
2 years ago
- Owner set to peterwilsoncc
- Resolution set to fixed
- Status changed from new to closed
In 55944:
Trac ticket: https://core.trac.wordpress.org/ticket/58551
Adds support for the CSS
repeatfunction.The Trac ticket mentions support for
auto-fitand-fill, but in testing I realised those are already implicitly supported so no changes needed there.To test this change in the front end, apply the patch on top of #4625 and follow testing steps from that PR.