Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#54929 closed defect (bug) (fixed)

Twenty Twenty-Two: Pricing Table pattern header levels should be consistent

Reported by: kjellr Owned by: audrasjb
Priority: normal Milestone: 5.9.1
Component: Bundled Theme Version: 5.9
Severity: normal Keywords: has-patch fixed-major
Cc: Focuses:

Description

The heading levels in the "Pricing Table" pattern bundled with Twenty Twenty-Two are not consistent.

"Pigeon" is a H3, but the other two are H2s. These should all be H3s.

---

Originally reported by @bobbingwide over in GitHub:
​https://github.com/WordPress/twentytwentytwo/issues/341

Attachments (1)

150486699-68fc52eb-19cc-4cbf-9657-f6015159929d.png​ (101.2 KB ) - added by kjellr 5 years ago.

Download all attachments as: .zip

Change History (8)

This ticket was mentioned in ​PR #2229 on ​WordPress/wordpress-develop by ​jffng.


5 years ago
#1

  • Keywords has-patch added; needs-patch removed

Fixes https://core.trac.wordpress.org/ticket/54929 and https://core.trac.wordpress.org/ticket/54928.

To test, check insert the Pricing Table pattern and verify the Heading levels are correct:

https://i0.wp.com/user-images.githubusercontent.com/5375500/151209104-00fdb1fe-81e2-4335-b5a5-13b6458c7869.png

Also visit the 404 template and verify the search block renders correctly.

​audrasjb commented on ​PR #2229:


5 years ago
#2

I removed the changes related to hidden-404 pattern since this has already been fixed earlier :)

#3 @audrasjb
5 years ago

  • Owner set to audrasjb
  • Resolution → fixed
  • Status new → closed

In 52681:

Twenty Twenty-Two: Fix headings consistency in Pricing table pattern.

This change fixes a consistency issue where "Pigeon" was a h3, while "Sparrow" and "Falcon" were h2. Let's treat these birds equally 🐦

Props kjellr, jffng.
Fixes #54929.

#5 @audrasjb
5 years ago

  • Keywords fixed-major added
  • Resolution fixed
  • Status closed → reopened

Reopening for 5.9 backport.

#6 @SergeyBiryukov
5 years ago

  • Summary [Twenty Twenty-Two] Pricing Table pattern header levels should be consistent → Twenty Twenty-Two: Pricing Table pattern header levels should be consistent

#7 @SergeyBiryukov
5 years ago

  • Resolution → fixed
  • Status reopened → closed

In 52711:

Twenty Twenty-Two: Fix headings consistency in Pricing table pattern.

This change fixes a consistency issue where "Pigeon" was a h3, while "Sparrow" and "Falcon" were h2. Let's treat these birds equally 🐦

Props kjellr, jffng, audrasjb.
Merges [52681] to the 5.9 branch.
Fixes #54929.

Note: See TracTickets for help on using tickets.