Skip to content

feat(ui): convert topup & raise modals to inline cards (v1.14) - #38

Merged
argszero merged 1 commit into
mainfrom
feat/ui-polish-inline-topup-raise
Aug 17, 2026
Merged

argszero merged 1 commit into
mainfrom
feat/ui-polish-inline-topup-raise

Conversation

@argszero

Copy link
Copy Markdown
Owner

Summary

Convert the remaining P1 modals (topup-modal, raise-modal) to inline expanding cards on the Wallet page, per the host's UI/UE polish instruction (rant 2026-08-17T15:50:05, item B.7 "少弹窗改造(优先)").

The wallet "充值 / 申请加额" buttons now expand a card in place (#topup-card / #raise-card), following the existing #share-form-card / #dept-form-card inline pattern:

  • Preset amounts (100/500/1000) + custom amount + confirm/cancel behavior kept identical
  • Topup and raise cards are mutually exclusive (opening one collapses the other)
  • Modal overlays, the generic data-modal-close binding, and the now-unused .modal-actions CSS rule removed
  • Remaining modal: P2 chat-modal (chat consumption) only — documented as future work

Also syncs docs: user-stories.md → v1.14 (§1.1 marks P1 done), ui/README.md backlog updated.

Related Issue

(no issue — host rant driven: 2026-08-17T15:50:05)

Changes

  • 充值(topup-modal → #topup-card 行内卡片)、申请加额(raise-modal → #raise-card 行内卡片)
  • 涉及文档同步:docs/user-stories.md v1.14、ui/README.md

Tests

  • cargo test 全部通过(1 passed)
  • cargo fmt --check 通过(无 Rust 改动)
  • UI 自测:node --check 通过;全部 $("#id") 引用与 HTML id 交叉校验无缺失;无残留 modal 引用

Checklist

  • 分支命名符合约定(feat/ui-polish-inline-topup-raise)
  • Commit message 使用 Conventional Commits 格式
  • 单一职责,改动最小化

Replace the remaining P1 modals (topup-modal, raise-modal) with inline
expanding cards on the wallet page (#topup-card / #raise-card), following
the established #share-form-card / #dept-form-card pattern:
- wallet buttons expand the card in place; topup and raise cards are
  mutually exclusive (opening one collapses the other)
- preset amounts + custom amount + confirm/cancel kept identical
- remove the modal overlays, data-modal-close binding, and the now-unused
  .modal-actions CSS rule
- docs: user-stories v1.14 §1.1 P1 done (only P2 chat-modal remains)
- rant 2026-08-17T15:50:05 (UI polish & UE, item B.7 modal→inline first)
@argszero
argszero merged commit bb6c547 into main Aug 17, 2026
1 check passed
@argszero
argszero deleted the feat/ui-polish-inline-topup-raise branch August 17, 2026 07:53
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