Skip to content

feat: expandable detail panels on summary screen - #2

Merged
KasperRagnar merged 1 commit into
masterfrom
feature/summary-details
Apr 23, 2026
Merged

KasperRagnar merged 1 commit into
masterfrom
feature/summary-details

Conversation

@KasperRagnar

Copy link
Copy Markdown
Owner

What changed

Clicking a summary tile now reveals a detail panel inline:

  • Files copied → breakdown by type (Photos / Videos / Music / Files)
  • Files skipped → list of skipped filenames
  • Errors → filename + full error message, stacked and scrollable

Clicking the same tile again collapses the panel. Clicking a different tile switches to its content. The panel is reset when returning to the start screen.

Why

The previous summary showed only three counts with no way to inspect what was behind each number. This makes it easy to see exactly which files failed or were skipped without needing to look at any logs.

Changes

  • TransferService.ts — SummaryEvent extended with copiedByType, skippedFiles, errorDetails
  • index.html — tiles made clickable; detail panel added
  • style.css — hover/active tile styles, detail panel and row styles
  • app.js — toggleDetailPanel, renderDetail, esc helpers; reset on back
  • i18n/ — new summary.images/videos/music/files/noDetails keys (en/da/es)
  • docs/screenshots/ — updated to show all new summary states; setup split into two screenshots

🤖 Generated with Claude Code

Clicking a summary tile now reveals contextual detail:
- Files copied → breakdown by type (photos, videos, music, files)
- Files skipped → list of skipped filenames
- Errors → filename + full error message, stacked and scrollable

TransferService.SummaryEvent extended with copiedByType, skippedFiles,
and errorDetails; all three populated during the transfer loop.
Screenshots updated to show the new summary states.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@KasperRagnar
KasperRagnar merged commit ee75f5e into master Apr 23, 2026
1 check passed
@KasperRagnar
KasperRagnar deleted the feature/summary-details branch April 23, 2026 20:06
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