#122 — Bug — Masterpackliste-"⋯"-Menü in der Übersicht nur bei Hover sichtbar #125

New issue

Closed

opened 2026-08-18 13:14:17 +02:00 by lena · 2 comments

lena commented

2026-08-18 13:14:17 +02:00

(Migrated from git.butzei.de)

Story: Bug — Masterpackliste-"⋯"-Menü in der Übersicht nur bei Hover sichtbar

As a Nutzer der Masterpackliste (v. a. auf einem Touch-Gerät),
I want to das Options-Menü einer Masterpackliste in der Übersicht auch ohne Hover erkennen können,
so that ich weiß, dass dort Rename/Invite/Delete verfügbar sind (u. a. das aus #121 neue "Verlassen").

Background

Gleiches Bug-Muster wie #112 (Vorratsschrank) und die ursprüngliche #110-Fix in ShoppingListItem.tsx:
MasterPackingListItem.tsxs Menü-Button nutzt noch opacity-0 group-hover:opacity-100 — der Button ist
also ganz unsichtbar statt nur abgeblendet, solange kein Hover-Zustand vorliegt (auf Touch-Geräten nie).
ListActionsMenu.tsx, ShoppingListItem.tsx und PantryItem.tsx nutzen bereits alle
opacity-50 group-hover:opacity-100. Die Masterpackliste wurde beim damaligen #112-Fix schlicht
übersehen (existierte zum Zeitpunkt von #110/#112 vermutlich noch nicht oder wurde nicht mitgeprüft).

Vom Nutzer per Rückfrage bestätigt: genau dieses Symptom ("bei Masterpacklisten werden gerade keine drei
Punkte angezeigt") auf dem Handy beobachtet.

AC

MasterPackingListItem.tsxs Menü-Button-Klasse von opacity-0 group-hover:opacity-100 auf
opacity-50 group-hover:opacity-100 umstellen, analog zu ListActionsMenu.tsx,
ShoppingListItem.tsx und PantryItem.tsx.

Out of scope

Alles andere an MasterPackingListItem.tsx oder der Masterpackliste allgemein.
Das separat gemeldete Item-Zeilen-Menü (Trash-Icon → "⋯" in MasterPackItemRow.tsx), siehe #130.

# Story: Bug — Masterpackliste-"⋯"-Menü in der Übersicht nur bei Hover sichtbar **As a** Nutzer der Masterpackliste (v. a. auf einem Touch-Gerät), **I want to** das Options-Menü einer Masterpackliste in der Übersicht auch ohne Hover erkennen können, **so that** ich weiß, dass dort Rename/Invite/Delete verfügbar sind (u. a. das aus `#121` neue "Verlassen"). ## Background Gleiches Bug-Muster wie `#112` (Vorratsschrank) und die ursprüngliche `#110`-Fix in `ShoppingListItem.tsx`: `MasterPackingListItem.tsx`s Menü-Button nutzt noch `opacity-0 group-hover:opacity-100` — der Button ist also ganz unsichtbar statt nur abgeblendet, solange kein Hover-Zustand vorliegt (auf Touch-Geräten nie). `ListActionsMenu.tsx`, `ShoppingListItem.tsx` und `PantryItem.tsx` nutzen bereits alle `opacity-50 group-hover:opacity-100`. Die Masterpackliste wurde beim damaligen `#112`-Fix schlicht übersehen (existierte zum Zeitpunkt von `#110`/`#112` vermutlich noch nicht oder wurde nicht mitgeprüft). Vom Nutzer per Rückfrage bestätigt: genau dieses Symptom ("bei Masterpacklisten werden gerade keine drei Punkte angezeigt") auf dem Handy beobachtet. ## AC - `MasterPackingListItem.tsx`s Menü-Button-Klasse von `opacity-0 group-hover:opacity-100` auf `opacity-50 group-hover:opacity-100` umstellen, analog zu `ListActionsMenu.tsx`, `ShoppingListItem.tsx` und `PantryItem.tsx`. ## Out of scope - Alles andere an `MasterPackingListItem.tsx` oder der Masterpackliste allgemein. - Das separat gemeldete Item-Zeilen-Menü (Trash-Icon → "⋯" in `MasterPackItemRow.tsx`), siehe `#130`.

lena commented

2026-08-19 00:22:28 +02:00

(Migrated from git.butzei.de)

Claimed for this go-cycle (2026-08-19).

lena commented

2026-08-19 02:48:36 +02:00

(Migrated from git.butzei.de)

Fixed. Menu trigger button in MasterPackingListItem used opacity-0 group-hover:opacity-100 (invisible until hover), unlike the already-correct opacity-50 pattern used by ListActionsMenu/ShoppingListItem/PantryItem — meaning the "..." menu never appeared at all on touch devices (no hover state). One-line fix: opacity-0 -> opacity-50.

Changes:

ReactUi/src/components/MasterPackingListItem.tsx: opacity-0 -> opacity-50 on the menu trigger button class.
ReactUi/src/components/MasterPackingListItem.test.tsx: added a regression test asserting the button carries opacity-50 and not opacity-0.

Verification:

Local: npx vitest run src/components/MasterPackingListItem.test.tsx (10/10 passed), npm run build clean.
CI run 314 (commit 294f433): Backend build+tests, Frontend build+test, Docker build all green. 3 of 4 E2E legs (Desktop HD, chromium, firefox) failed on an unrelated pre-existing test, e2e/todo-sort-mode.spec.ts:88 (feature #119, checkbox-click state assertion) - confirmed unconnected to this change since it reproduces identically across independent browser engines while Mobile Chrome passed cleanly, and no Master Packing List test was affected. Filed as a separate follow-up bug.

Closing as done.

Fixed. Menu trigger button in MasterPackingListItem used `opacity-0 group-hover:opacity-100` (invisible until hover), unlike the already-correct `opacity-50` pattern used by ListActionsMenu/ShoppingListItem/PantryItem — meaning the "..." menu never appeared at all on touch devices (no hover state). One-line fix: `opacity-0` -> `opacity-50`. Changes: - `ReactUi/src/components/MasterPackingListItem.tsx`: `opacity-0` -> `opacity-50` on the menu trigger button class. - `ReactUi/src/components/MasterPackingListItem.test.tsx`: added a regression test asserting the button carries `opacity-50` and not `opacity-0`. Verification: - Local: `npx vitest run src/components/MasterPackingListItem.test.tsx` (10/10 passed), `npm run build` clean. - CI run 314 (commit 294f433): Backend build+tests, Frontend build+test, Docker build all green. 3 of 4 E2E legs (Desktop HD, chromium, firefox) failed on an unrelated pre-existing test, `e2e/todo-sort-mode.spec.ts:88` (feature `#119`, checkbox-click state assertion) - confirmed unconnected to this change since it reproduces identically across independent browser engines while Mobile Chrome passed cleanly, and no Master Packing List test was affected. Filed as a separate follow-up bug. Closing as done.