Opened 2 months ago
Last modified 2 months ago
#53797 new defect (bug)
Bundled Themes: Add "Tested up to" in style.css
Reported by: | kafleg | Owned by: | |
---|---|---|---|
Milestone: | 5.9 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | has-patch |
Focuses: | Cc: |
Description
Twenty Twenty theme is missing Tested up to in style.css and this is the required thing. I have added Tested up to: 5.8
in Github.
Attachments (2)
Change History (10)
This ticket was mentioned in PR #1521 on WordPress/wordpress-develop by kafleg.
2 months ago
- Keywords has-patch added; needs-patch removed
This ticket was mentioned in PR #1521 on WordPress/wordpress-develop by kafleg.
2 months ago
Tested up to is required and it needs to be mentioned in style.css
Trac ticket: https://core.trac.wordpress.org/ticket/53797
#3
@
2 months ago
- Component changed from Themes to Bundled Theme
- Keywords commit added
- Version 5.8 deleted
Hi there!
Thanks for the ticket and PR. It looks fine.
Marking for commit
.
#5
@
2 months ago
- Keywords commit removed
Related: #53276, #52982, #51501, #50503, #48517, #53580, meta-461.
Thanks for this, @kafleg!
I've taken a closer look, and it appears that Twenty Twenty-One is the only bundled theme that currently has Tested up to
in style.css
headers. This was an intentional decision at the time because WordPress Core did not yet utilize this header (see comment:5:ticket:48517).
Since then, [47145] added support for required WordPress versions in Core, but Tested up to
is still not supported by Core within style.css
headers.
I was not able to locate a ticket in Trac to consider adding this support to style.css
files, so this could serve as that. But more work would be required, as Core itself must have a reason to add support for this header.
An example of this could be adding "This theme has not been marked as compatible with your version of WordPress." messages to match those shown when working with plugins.
Twenty Twenty theme is missing Tested up to in style.css and this is the required thing.
Can you expand on the "required thing"? Is this now enforced for themes residing on WordPress.org? If I recall correctly, that was not the case previously (it was a soft requirement/suggestion).
#6
@
2 months ago
@desrosj Yes, it was recommended before. But, while reviewing themes we used to ask theme authors to add them. But when the Theme Check plugin is updated to the newer version, Tested up to
is now REQUIRED. You can also see the Requirements handbook about this.
Tested up to is required and it needs to be mentioned in style.css
Trac ticket: https://core.trac.wordpress.org/ticket/53797