---
phase: 35-bug-fixes
plan: "03"
subsystem: client-ui
tags: [cursor, ux, bug-fix, FIX-05]
dependency_graph:
  requires: []
  provides: [cursor-pointer-audit]
  affects: [ItemCard, FabMenu, BottomTabBar]
tech_stack:
  added: []
  patterns: [conditional-tailwind-class, explicit-cursor-pointer]
key_files:
  modified:
    - src/client/components/ItemCard.tsx
    - src/client/components/FabMenu.tsx
    - src/client/components/BottomTabBar.tsx
decisions:
  - "Add cursor-pointer explicitly to each interactive element rather than relying on browser defaults"
  - "Biome formatter requires multi-line attribute splitting for button elements with 3+ attributes"
metrics:
  duration: "~5 minutes"
  completed: "2026-04-19"
  tasks_completed: 2
  files_modified: 3
---

# Phase 35 Plan 03: Cursor-Pointer Audit Summary

Explicit `cursor-pointer` added to all interactive elements that lacked it — ItemCard navigable outer button, FabMenu menu item buttons and main FAB, and BottomTabBar's three anonymous tab buttons. Resolves FIX-05.

## Tasks Completed

| Task | Name | Commit | Files |
|------|------|--------|-------|
| 1 | Add cursor-pointer to ItemCard navigable case | e1d516c | src/client/components/ItemCard.tsx |
| 2 | Add cursor-pointer to FabMenu and BottomTabBar buttons | d58f7fa | src/client/components/FabMenu.tsx, src/client/components/BottomTabBar.tsx |

## What Was Built

### Task 1 — ItemCard (FIX-05)

The outer `<button>` in `ItemCard.tsx` had a conditional className: `cursor-default` when `linkTo === null` (setup cards, non-navigable), but was missing `cursor-pointer` in the non-null branch (collection cards, navigable). Added `cursor-pointer` to the non-null branch. The `cursor-default` branch is preserved unchanged.

### Task 2 — FabMenu and BottomTabBar (FIX-05)

FabMenu: Added `cursor-pointer` to both `motion.button` elements — the menu item buttons rendered per `menuItems` array, and the main FAB toggle button.

BottomTabBar: Added `cursor-pointer` to all three anonymous user `<button>` elements — the collection tab, setups tab, and search tab. Biome formatter required multi-line attribute expansion (type, onClick, className each on their own line) to pass lint.

## Deviations from Plan

### Auto-fixed Issues

**1. [Rule 1 - Format] Biome formatter required multi-line button attribute splitting**
- **Found during:** Task 2 verification (lint)
- **Issue:** Biome's formatter rejected single-line `<button type="button" onClick={...} className="cursor-pointer">` — three attributes triggered multi-line expansion requirement
- **Fix:** Split all three BottomTabBar button elements to multi-line format matching Biome's output
- **Files modified:** src/client/components/BottomTabBar.tsx
- **Commit:** d58f7fa (included in same task commit)

## Verification

- `grep -n "cursor-pointer hover:border-gray-200" ItemCard.tsx` — 1 match on outer button (confirmed)
- `grep -n "cursor-default" ItemCard.tsx` — 1 match on null linkTo branch (preserved)
- `grep -c "cursor-pointer" FabMenu.tsx` — 2 (menu item button + FAB button)
- `grep -c "cursor-pointer" BottomTabBar.tsx` — 3 (collection, setups, search tabs)
- `bun run lint` — passes (0 errors, 1 pre-existing warning in scripts/)
- `bun test` — 464 pass, 0 fail

## Known Stubs

None.

## Threat Flags

None — pure CSS class changes, no logic, data flow, or auth boundary touched.

## Self-Check: PASSED

- e1d516c exists: FOUND
- d58f7fa exists: FOUND
- src/client/components/ItemCard.tsx: FOUND
- src/client/components/FabMenu.tsx: FOUND
- src/client/components/BottomTabBar.tsx: FOUND
