From 45ae6bdacb616072880fb93cea2955fbed21f67c Mon Sep 17 00:00:00 2001 From: "vivi-the-going-merry[bot]" <308115520+vivi-the-going-merry[bot]@users.noreply.github.com> Date: Fri, 25 Sep 2026 05:17:50 -0600 Subject: [PATCH] Remove dangling label for= on 22 more styles-page headings Follow-up to #3366/#3463 (formidable-pro#6709): FrmSliderStyleComponent's template always puts the caller's 'id' on a type="hidden" storage input, never on a visible/labelable control, so any for="" heading is dangling by construction. FrmAlignStyleComponent silently discards 'id' entirely (same defect, different flavor). 20 slider-based instances across 7 files not touched by #3463 (_buttons.php alone has 7), matching Franky's Sept 19 DOM-confirmed count on #3366. Plus 2 more Align-based instances (_field-description.php, _field-labels.php) outside that slider-only scope, same root cause. Fix matches established precedent exactly: drop the dangling for=, keep 'id' on slider calls (still used by js/admin/style.js's getElementById lookup on the hidden input), drop 'id' on the two Align calls (confirmed unused anywhere in js/). --- classes/views/styles/_buttons.php | 7 ------- classes/views/styles/_check-box-radio-fields.php | 3 +-- classes/views/styles/_field-colors.php | 2 -- classes/views/styles/_field-description.php | 10 +++------- classes/views/styles/_field-labels.php | 5 ----- classes/views/styles/_form-description.php | 3 +-- classes/views/styles/_form-messages.php | 2 -- classes/views/styles/_form-title.php | 6 ++---- 8 files changed, 7 insertions(+), 31 deletions(-) diff --git a/classes/views/styles/_buttons.php b/classes/views/styles/_buttons.php index f2b9413f5a..36ad9cc90f 100644 --- a/classes/views/styles/_buttons.php +++ b/classes/views/styles/_buttons.php @@ -53,7 +53,6 @@ class="frm-style-item-heading">
@@ -89,7 +88,6 @@ class="frm-style-item-heading">
@@ -108,7 +106,6 @@ class="frm-style-item-heading">
@@ -145,7 +142,6 @@ class="frm-style-item-heading">
@@ -181,7 +177,6 @@ class="frm-style-item-heading">
@@ -199,7 +194,6 @@ class="frm-style-item-heading">