stackchain/stackchain-dashboard
1
0
Fork 0
Code Issues 9 Pull Requests 1 Actions Packages Projects Releases 713 Wiki Activity

Review and caption mobile conversation photo bundles before sending #945

New Issue
Closed
opened 2026-08-16 06:50:58 +00:00 by rockachopa · 0 comments
rockachopa commented 2026-08-16 06:50:58 +00:00
Member
Copy Link

User problem

Mobile issue, pull-request, and unread-update conversation composers can send up to five photos, but show only the latest preview. Operators cannot review the whole bundle, correct evidence order, remove an arbitrary bad photo, or caption what each photo proves before sending or queueing offline.

Evidence

  • frontend/dashboard.js mounts all three conversation attachment controllers with maxFiles: 5 but without the reusable tray/reorder/note controls.
  • frontend/index.html exposes only one preview and “Remove screenshot” in those composers, while New issue already has a tested bundle-review flow.
  • frontend/authored-outbox.js drops per-photo notes even though background delivery already renders attachment notes safely.
  • No open issue covers conversation photo review/captions; historical #825/#827 are explicitly limited to New issue filing, while #941/#943 only capture and deliver conversation photos.

Acceptance criteria

  • Issue comments, pull comments, and update replies show every selected photo in a touch-scrollable review tray.
  • The selected photo can be moved earlier/later, removed independently, and given a bounded caption that stays paired through reorder/removal.
  • Foreground and durable offline delivery preserve visible order and captions; legacy note-less records still work and binary bytes remain IndexedDB-only.
  • At 320px and 390px mobile widths there is no page-level horizontal overflow and actionable controls retain 44px touch targets.
  • Behavioral tests prove all three mounts, note persistence, ordered Markdown delivery, and mobile layout; full suite and release E2E remain green.
## User problem Mobile issue, pull-request, and unread-update conversation composers can send up to five photos, but show only the latest preview. Operators cannot review the whole bundle, correct evidence order, remove an arbitrary bad photo, or caption what each photo proves before sending or queueing offline. ## Evidence - `frontend/dashboard.js` mounts all three conversation attachment controllers with `maxFiles: 5` but without the reusable tray/reorder/note controls. - `frontend/index.html` exposes only one preview and “Remove screenshot” in those composers, while New issue already has a tested bundle-review flow. - `frontend/authored-outbox.js` drops per-photo notes even though background delivery already renders attachment notes safely. - No open issue covers conversation photo review/captions; historical #825/#827 are explicitly limited to New issue filing, while #941/#943 only capture and deliver conversation photos. ## Acceptance criteria - Issue comments, pull comments, and update replies show every selected photo in a touch-scrollable review tray. - The selected photo can be moved earlier/later, removed independently, and given a bounded caption that stays paired through reorder/removal. - Foreground and durable offline delivery preserve visible order and captions; legacy note-less records still work and binary bytes remain IndexedDB-only. - At 320px and 390px mobile widths there is no page-level horizontal overflow and actionable controls retain 44px touch targets. - Behavioral tests prove all three mounts, note persistence, ordered Markdown delivery, and mobile layout; full suite and release E2E remain green.
timmy was assigned by rockachopa 2026-08-16 06:50:58 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
P0

Drop everything

  
P1

High priority

  
P2

Medium priority

  
agent:nightprowl

Assignments routed to the LongCat NightProwl evaluation worker

  
backend

Backend service

  
ci

Continuous integration / actions

  
frontend

Frontend / UI

  
meta

Meta / planning

  
pages

Gitea Pages

  
template

Repo template / standards

No Label
P0
P1
P2
agent:nightprowl
backend
ci
frontend
meta
pages
template
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
timmy
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: stackchain/stackchain-dashboard#945
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?