Skip to content

feat: work order list primary photo thumbnail (Story 17.11)#295

Merged
daveharmswebdev merged 2 commits intomainfrom
feature/17-11-wo-list-primary-photo-thumbnail
Mar 7, 2026
Merged

feat: work order list primary photo thumbnail (Story 17.11)#295
daveharmswebdev merged 2 commits intomainfrom
feature/17-11-wo-list-primary-photo-thumbnail

Conversation

@daveharmswebdev
Copy link
Owner

Summary

  • Add 48×48px photo thumbnail to each work order list row (40×40 on mobile)
  • Shows primary photo with lazy loading when available, handyman mat-icon placeholder otherwise
  • 6 new unit tests covering img rendering, src binding, lazy loading, placeholder icon, container count, and no-img-in-placeholder

Acceptance Criteria

  • AC-1: Thumbnail displayed in row between expand chevron and row content ✅
  • AC-2: Placeholder handyman icon for no-photo work orders ✅
  • AC-3: Backend already includes primaryPhotoThumbnailUrl — no changes needed ✅
  • AC-4: Mobile responsive (40×40px at ≤768px) ✅

Test plan

  • All 2625 frontend unit tests pass (0 regressions)
  • Verify thumbnail appears for work orders with photos
  • Verify handyman placeholder for work orders without photos
  • Verify mobile layout at ≤768px viewport

Closes #270

🤖 Generated with Claude Code

daveharmswebdev and others added 2 commits March 5, 2026 19:05
Add 48×48px photo thumbnail to each work order row between expand
chevron and row content. Shows primary photo with lazy loading when
available, falls back to handyman mat-icon placeholder. Responsive
to 40×40px on mobile (≤768px). Includes 6 unit tests.

Closes #270

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
… expand thumbnail

Wrap thumbnail + row-content in .row-main flex container to fix missing
gap (12px) and mobile layout (stays inline instead of stacking). Change
.wo-thumbnail background to surface-container-high for hover contrast.
Remove redundant expand panel thumbnail (row thumbnail now provides
visual identification). Remove 2 related tests.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@daveharmswebdev daveharmswebdev merged commit bc5cbb5 into main Mar 7, 2026
7 checks passed
@daveharmswebdev daveharmswebdev deleted the feature/17-11-wo-list-primary-photo-thumbnail branch March 7, 2026 14:13
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.

Work orders list: Display primary photo thumbnail (match Properties list)

1 participant