fix: improve mobile detail tab targets

This commit is contained in:
Peter Steinberger
2026-05-12 08:11:21 +01:00
parent 5b10c21b50
commit dded7a55b9
4 changed files with 7 additions and 5 deletions
+2
View File
@@ -6,6 +6,8 @@
### Fixes
- Web: keep skill/plugin detail tabs at mobile-friendly touch target height.
## 0.15.0 - 2026-05-12
### Changes
+1 -1
View File
@@ -52,7 +52,7 @@ test("known public skill detail links to owner profile", async ({ page, request
const errors = trackRuntimeErrors(page);
await page.goto(`/${ownerHandle}/${slug}`, { waitUntil: "domcontentloaded" });
const ownerLink = page.locator(".user-handle").first();
const ownerLink = page.locator(`a[href="/p/${ownerHandle}"]`).first();
await expect(ownerLink).toHaveAttribute("href", new RegExp(`/p/${ownerHandle}$`));
await ownerLink.click();
+3 -3
View File
@@ -86,7 +86,7 @@ test("skill detail page has no horizontal overflow on mobile", async ({ page, re
);
await page.goto(`/${ownerHandle}/${slug}`, { waitUntil: "domcontentloaded" });
await expect(page.getByRole("heading", { name: payload.skill!.displayName! })).toBeVisible();
await expect(page.locator("h1.skill-page-title")).toHaveText(payload.skill!.displayName!);
const scrollWidth = await page.evaluate(() => document.documentElement.scrollWidth);
const clientWidth = await page.evaluate(() => document.documentElement.clientWidth);
@@ -112,8 +112,8 @@ test("detail tabs are scrollable and touch-friendly on mobile", async ({ page, r
await page.goto(`/${ownerHandle}/${slug}`, { waitUntil: "domcontentloaded" });
// All standard tabs should be accessible (even if scrolled)
for (const tabName of ["README", "Files", "Versions"]) {
const tab = page.getByRole("button", { name: tabName });
for (const tabName of ["SKILL.md", "Files", "Versions"]) {
const tab = page.getByRole("tab", { name: tabName });
await tab.scrollIntoViewIfNeeded();
await expect(tab).toBeVisible();
+1 -1
View File
@@ -5470,7 +5470,7 @@ code {
font-weight: 600;
color: var(--ink-soft);
cursor: pointer;
min-height: 36px;
min-height: 44px;
white-space: nowrap;
flex-shrink: 0;
transform: translateY(1px);