#46213 closed defect (bug) (fixed)
Twenty Nineteen: Remove the incorrect "flexible-header" tag.
| Reported by: | poena | Owned by: | audrasjb |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.2 |
| Component: | Bundled Theme | Version: | |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: | Focuses: |
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
@
7 years ago
- Keywords needs-refresh added
- Milestone Awaiting Review → 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
@
4 years ago
- Keywords commit added
- Milestone Future Release → 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.
4 years ago
#5
#6
@
4 years ago
I added a PR because the tag also need to be removed from the readme file :)
Looks good to me, now.
@audrasjb commented on PR #3672:
4 years ago
#8
Committed in https://core.trac.wordpress.org/changeset/54869
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Patch.