Skip to content

fix: improve mobile purchase actions - #38

Merged
apesch85 merged 1 commit into
devfrom
fix/mobile-purchase-actions
Sep 2, 2026
Merged

fix: improve mobile purchase actions#38
apesch85 merged 1 commit into
devfrom
fix/mobile-purchase-actions

Conversation

@apesch85

@apesch85 apesch85 commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

What does this PR do?

  • Keeps the inventory row action menu inside the mobile viewport and expands the three-dot touch target.
  • Makes copy purchase-price editing full-width on mobile with a sticky save footer and explicit save-error handling.
  • Changes Field Mode Bought It into an explicit purchase confirmation that asks for price paid, quantity, and condition.
  • Clarifies the difference between an existing catalog/price record and an owned copy.

Testing

  • npm run lint
  • npx tsc --noEmit
  • npm test -- --run — 445/445 passed
  • npm run build
  • node scripts/release-smoke.mjs
  • Puppeteer mobile viewport: menu fully visible/fixed, edited price persisted in PUT payload, Field purchase price persisted in purchase payload

Type of Change

  • Bug fix

Reported by Ded.Gamer in Discord.

Keep inventory action menus inside the mobile viewport, make copy purchase-price editing touch friendly, and require an explicit paid-price confirmation in Field Mode. Clarify catalog-only records versus owned copies and add regressions for each reported flow.
@apesch85
apesch85 merged commit eab74ed into dev Sep 2, 2026
2 checks passed
@apesch85
apesch85 deleted the fix/mobile-purchase-actions branch September 2, 2026 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant