From 1e51ffbde07b133287bdbd54404119ede0383487 Mon Sep 17 00:00:00 2001 From: R-Hart80 Date: Mon, 13 Apr 2026 17:08:25 -0300 Subject: [PATCH 1/2] fix: dim labels and reposition note for webhook-driven readonly fields When a project is webhook-driven, the four GitHub integration input fields (repo owner, repo name, branch, file path) were already marked readOnly and visually styled as disabled on the inputs themselves. This commit extends the visual treatment to their labels by applying opacity-60 when isWebhookDriven is true, and moves the explanatory note ("Repository fields are managed by the GitHub integration.") to appear after all four fields so it clearly covers the whole group. Co-Authored-By: Claude Sonnet 4.6 --- app/projects/[id]/settings/page.tsx | 32 ++++++++++++++++++++--------- 1 file changed, 22 insertions(+), 10 deletions(-) diff --git a/app/projects/[id]/settings/page.tsx b/app/projects/[id]/settings/page.tsx index e420d8d4..0c846e41 100644 --- a/app/projects/[id]/settings/page.tsx +++ b/app/projects/[id]/settings/page.tsx @@ -2883,7 +2883,10 @@ function RemoteSyncSection({ {/* Repository */}
-
-
- {isWebhookDriven && ( -

- Repository fields are managed by the GitHub integration. -

- )} - {/* Same-repo info when editing form matches GitHub integration */} {!isWebhookDriven && githubIntegration && repoOwner === githubIntegration.repo_owner && @@ -2939,7 +2939,10 @@ function RemoteSyncSection({ {/* Branch + File path */}
-
-
+ {isWebhookDriven && ( +

+ Repository fields are managed by the GitHub integration. +

+ )} + {/* Frequency + Update mode */}
From f16652cd5f87d2a4837fcb9815596f59f30f256d Mon Sep 17 00:00:00 2001 From: "John R. D'Orazio" Date: Wed, 19 Aug 2026 01:24:51 +0200 Subject: [PATCH 2/2] revert(settings): keep webhook-driven field labels at full contrast MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Drop the `isWebhookDriven && "opacity-60"` additions on the four GitHub-integration field labels. These fields are read-only but *informational* — users are meant to read and confirm the real owner / name / branch / path. Labels are never interactive, so fading them communicates nothing functional and costs legibility. It is also a WCAG regression: the labels are `text-slate-700` (#334155) at `text-sm` / `font-medium`, i.e. normal text, so the 4.5:1 AA threshold applies. At full opacity they measure ~9.3:1; at `opacity-60` the effective color is ~#858d99, or ~3.35:1 — below AA. Dark mode (slate-300 on slate-800) takes a comparable hit. The note repositioning is kept: "Repository fields are managed by the GitHub integration." now sits after both field grids, so it reads as a caption for all four fields rather than only the bottom two. Refs #5 Co-Authored-By: R-Hart80 Co-Authored-By: Claude Opus 5 (1M context) --- app/projects/[id]/settings/page.tsx | 20 ++++---------------- 1 file changed, 4 insertions(+), 16 deletions(-) diff --git a/app/projects/[id]/settings/page.tsx b/app/projects/[id]/settings/page.tsx index 0c846e41..8d7e3dd3 100644 --- a/app/projects/[id]/settings/page.tsx +++ b/app/projects/[id]/settings/page.tsx @@ -2883,10 +2883,7 @@ function RemoteSyncSection({ {/* Repository */}
-
-