From aec9701656b03dc8092af53a9c705e3ee9ce8f74 Mon Sep 17 00:00:00 2001 From: Brandonma21 Date: Thu, 17 Sep 2026 10:57:09 +0200 Subject: [PATCH 1/3] Fix editor validation errors on Services pill/tag badges (LS-4207) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Bug fix - services-linked-decisions.php, services-service-clusters.php: add missing `has-border-color` class to the process-pill and cluster-tag wrapper groups - Root cause: both blocks set an inline border color via a custom JSON style attribute, but WordPress's border block support always expects a `has-border-color` class on the wrapper whenever a border color is set — the hand-authored static HTML never included it, so the editor's re-derived expected markup never matched the stored HTML, permanently flagging these blocks as "unexpected or invalid content" - Confirmed via a working control case in services-cta.php, which sets a border color the same way and already includes the class correctly --- patterns/sections/services-linked-decisions.php | 2 +- patterns/sections/services-service-clusters.php | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/patterns/sections/services-linked-decisions.php b/patterns/sections/services-linked-decisions.php index c27ba77..70b1d9c 100644 --- a/patterns/sections/services-linked-decisions.php +++ b/patterns/sections/services-linked-decisions.php @@ -99,7 +99,7 @@ ?> -
+

diff --git a/patterns/sections/services-service-clusters.php b/patterns/sections/services-service-clusters.php index 895fda0..50c8401 100644 --- a/patterns/sections/services-service-clusters.php +++ b/patterns/sections/services-service-clusters.php @@ -133,7 +133,7 @@ -
+
From e123ea2c48c17715d6be7233d83a8c0b7dc814b4 Mon Sep 17 00:00:00 2001 From: Brandonma21 Date: Thu, 17 Sep 2026 11:22:08 +0200 Subject: [PATCH 2/3] Add CHANGELOG entry for Services badge border-color fix (LS-4207) Docs - CHANGELOG.md: add dated entry for the has-border-color fix on services-linked-decisions.php and services-service-clusters.php, matching the one-entry-per-PR convention already used for the other Services page and mobile-menu entries --- CHANGELOG.md | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 3e8908d..3c5f14d 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 --- +## [Unreleased] — Fix editor validation errors on Services pill/tag badges (LS-4207) + +### Fixed + +- `patterns/sections/services-linked-decisions.php`, `patterns/sections/services-service-clusters.php`: added the missing `has-border-color` class to the `ls-process-pill` and `ls-cluster-tag` wrapper groups. Both blocks set an inline border colour via a custom JSON style attribute, but WordPress's border block support always expects a `has-border-color` class on the wrapper whenever a border colour is set. The hand-authored static HTML never included it, so the editor's re-derived expected markup never matched the stored HTML, permanently flagging these blocks as "Block contains unexpected or invalid content" with an "Attempt Recovery" prompt in the Site Editor. + +--- + ## [Unreleased] — Fix mobile menu: restore dropdown links, remove Systems, refine spacing (LS-3222) ### Fixed From b5e7a1597db8d1fefb828ecb0057e0ad69f149c2 Mon Sep 17 00:00:00 2001 From: Brandonma21 Date: Thu, 17 Sep 2026 11:26:21 +0200 Subject: [PATCH 3/3] Link PR #59 in CHANGELOG entry for LS-4207 Docs - CHANGELOG.md: add PR reference link now that the pull request exists, matching the existing per-PR link convention Co-Authored-By: Claude Sonnet 5 --- CHANGELOG.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 3c5f14d..7f75150 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -13,6 +13,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 - `patterns/sections/services-linked-decisions.php`, `patterns/sections/services-service-clusters.php`: added the missing `has-border-color` class to the `ls-process-pill` and `ls-cluster-tag` wrapper groups. Both blocks set an inline border colour via a custom JSON style attribute, but WordPress's border block support always expects a `has-border-color` class on the wrapper whenever a border colour is set. The hand-authored static HTML never included it, so the editor's re-derived expected markup never matched the stored HTML, permanently flagging these blocks as "Block contains unexpected or invalid content" with an "Attempt Recovery" prompt in the Site Editor. +([#59](https://github.com/lightspeedwp/ls-theme/pull/59)) + --- ## [Unreleased] — Fix mobile menu: restore dropdown links, remove Systems, refine spacing (LS-3222)