#46213 closed defect (bug) (fixed)
Twenty Nineteen: Remove the incorrect "flexible-header" tag.
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.2 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | has-patch commit |
Focuses: | Cc: |
Description
In style.css, the tag flexible-header is included.
But the requirements for using this tag is that the theme includes
support for a custom header with the flex-height and/or flex-width parameters.
The theme does not have support for a header image.
Ref: https://make.wordpress.org/themes/handbook/review/required/theme-tags/
Attachments (2)
Change History (10)
#2
@
6 years ago
- Keywords needs-refresh added
- Milestone changed from Awaiting Review to Future Release
Thanks for the report, @poena, and for the patch, @mukesh27!
For the patch, the generated RTL styles also includes a list of the tags -- for completeness, could you also please remove it there? Thanks!
#4
@
2 years ago
- Keywords commit added
- Milestone changed from Future Release to 6.2
I can confirm that the patch removes the incorrect tag.
This ticket was mentioned in PR #3672 on WordPress/wordpress-develop by @audrasjb.
2 years ago
#5
#6
@
2 years ago
I added a PR because the tag also need to be removed from the readme file :)
Looks good to me, now.
#7
@
2 years ago
- Owner set to audrasjb
- Resolution set to fixed
- Status changed from new to closed
In 54869:
@audrasjb commented on PR #3672:
2 years ago
#8
Committed in https://core.trac.wordpress.org/changeset/54869
Patch.