From 64186bd8decbe340f1f7e66ce3c1666a8a1a7d6e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= <mbombeck@gmail.com>
Date: Sun, 17 May 2026 14:52:48 +0200
Subject: [PATCH] test(e2e): update settings touch-target assertions to the
 v1.4.34.5 44 px floor
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The v1.4.34.5 iOS-textarea-zoom sweep lifted the mobile input
height floor from 40 px (`h-10`) to 44 px (`h-11`) on `Input`,
`Select` trigger, and the native `<select>` primitives — the
desktop tier still resolves to 40 px via `sm:h-10`. The
settings-mobile-consistency spec still pinned every form input,
the Compare-to trigger, and every `/settings/ai` native select to
40 px, so it went red on the v1.4.36 tag while the implementation
was correct.

Bump the three height assertions to 44 px and refresh the header
docblock so a future maintainer reads the contract as "44 on
mobile, 40 on >=sm" instead of stumbling on a stale baseline.
---
 e2e/settings-mobile-consistency.spec.ts | 20 +++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/e2e/settings-mobile-consistency.spec.ts b/e2e/settings-mobile-consistency.spec.ts
index ec83ded5b..062bf9e38 100644
--- a/e2e/settings-mobile-consistency.spec.ts
+++ b/e2e/settings-mobile-consistency.spec.ts
@@ -8,10 +8,16 @@ import { STORAGE_STATE_PATH } from "./setup/global-setup";
  * (`h-10`) on `Input`, `Select` trigger, and the native `<select>`
  * primitives per the WCAG 2.5.5 tap-target sweep (MB2). The
  * Dashboard Compare-to trigger followed the same path.
+ * v1.4.34.5 — mobile input floor lifted again from 40 px (`h-10`) to
+ * 44 px (`h-11`) to clear the WCAG 2.5.5 touch-target minimum on
+ * iOS Safari (textarea-zoom sweep). The `sm:h-10` desktop tier is
+ * unchanged; this spec runs at the Pixel 5 viewport so the mobile
+ * 44 px floor is what we lock in.
  *
  * The fixes that still apply:
  *
- *   - 40 px (`h-10`) is the canonical input height across Settings.
+ *   - 44 px (`h-11` on mobile, `sm:h-10` on >=sm) is the canonical
+ *     input height across Settings.
  *   - Title + action rows use `flex-col` on `<sm` and `flex-row` on
  *     `>=sm` so the action button stacks below on mobile.
  *   - Sprache pairs with date-of-birth in a single `sm:grid-cols-2`
@@ -28,7 +34,7 @@ test.describe("Settings mobile consistency (Pixel 5)", () => {
     test.skip(testInfo.project.name !== "chromium-mobile", "mobile-only spec");
   });
 
-  test("/settings/account: every form input renders at 40 px", async ({
+  test("/settings/account: every form input renders at 44 px", async ({
     page,
   }) => {
     await page.goto("/settings/account", { waitUntil: "networkidle" });
@@ -50,7 +56,7 @@ test.describe("Settings mobile consistency (Pixel 5)", () => {
 
     expect(formInputs.length).toBeGreaterThan(0);
     for (const inp of formInputs) {
-      expect.soft(inp.height, `${inp.tag}#${inp.id} (${inp.type})`).toBe(40);
+      expect.soft(inp.height, `${inp.tag}#${inp.id} (${inp.type})`).toBe(44);
     }
   });
 
@@ -132,7 +138,7 @@ test.describe("Settings mobile consistency (Pixel 5)", () => {
     );
   });
 
-  test("/settings/dashboard: Compare-to trigger renders at 40 px", async ({
+  test("/settings/dashboard: Compare-to trigger renders at 44 px", async ({
     page,
   }) => {
     await page.goto("/settings/dashboard", { waitUntil: "networkidle" });
@@ -141,10 +147,10 @@ test.describe("Settings mobile consistency (Pixel 5)", () => {
     const trigger = page.locator("#comparison-baseline");
     await expect(trigger).toBeVisible();
     const h = await trigger.evaluate((el) => el.getBoundingClientRect().height);
-    expect(Math.round(h)).toBe(40);
+    expect(Math.round(h)).toBe(44);
   });
 
-  test("/settings/ai: every native select renders at 40 px", async ({
+  test("/settings/ai: every native select renders at 44 px", async ({
     page,
   }) => {
     await page.goto("/settings/ai", { waitUntil: "networkidle" });
@@ -158,7 +164,7 @@ test.describe("Settings mobile consistency (Pixel 5)", () => {
 
     expect(heights.length).toBeGreaterThan(0);
     for (const h of heights) {
-      expect.soft(h).toBe(40);
+      expect.soft(h).toBe(44);
     }
   });
 });