Skip to content

[T3304] FIX: lazy-load letter previews, reveal shell on DOMContentLoaded - #360

Merged
Danielgergely merged 1 commit into
18.0from
T3304-letters-lazy-preview
Aug 5, 2026
Merged

[T3304] FIX: lazy-load letter previews, reveal shell on DOMContentLoaded#360
Danielgergely merged 1 commit into
18.0from
T3304-letters-lazy-preview

Conversation

@Danielgergely

Copy link
Copy Markdown
Member
  • FIX: reduces loading time of the letter page

@greptile-apps

greptile-apps Bot commented Aug 5, 2026

Copy link
Copy Markdown

Confidence Score: 5/5

The updated letter-preview and Capacitor visibility behavior is safe to merge based on the exercised browser and DOM interactions.

No defects remain. The changed lazy-loading, already-read click, and Capacitor reveal paths were executed successfully.

Files Needing Attention: No files need follow-up attention.

T-Rex T-Rex Logs

What T-Rex did

  • Ran focused jsdom and headless Chromium validations against the changed letter-preview JavaScript, letter-card markup and styling, and Capacitor visibility script.
  • Verified the read-letter iframe size inside the envelope (90.71 × 15 px) and confirmed data-src promotion to src when entering the configured 300 px root margin, and that clicking an already-read envelope navigates to the PDF URL through the standard interaction path.
  • Confirmed normal DOMContentLoaded handling and a late-listener simulation that restores document opacity to 1.
  • Validated that iframe observability remained functional and the already-read click navigated to the intended PDF URL; observed normal DOMContentLoaded and timeout fallback, and noted that an authored runnable test source and its captured output were uploaded.

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (1): Last reviewed commit: "[T3304] FIX: lazy-load letter previews, ..." | Re-trigger Greptile

@Danielgergely
Danielgergely merged commit 41a0b70 into 18.0 Aug 5, 2026
2 checks passed
@Danielgergely
Danielgergely deleted the T3304-letters-lazy-preview branch August 5, 2026 08:44
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