Adds estimated export time and file size indicators directly under the export buttons to resolve #224.
Changes
- Added size (MB) and duration (sec/min) estimation text below the main export button in
ExportPage.tsx.
- Added estimated size and time badge to thumbnail hover cards in
recordings-overlay.tsx.
Tested locally with short and long recording lengths.
/claim #224
Greptile Summary
This PR adds estimated export size and duration text to the editor export action and recording-thumbnail hover cards.
Confidence Score: 4/5
The PR appears safe to merge, with a non-blocking readability issue for long duration estimates in the recording overlay.
The estimate units and guarded data access are consistent with their producers, but the overlay’s seconds-only formatting becomes difficult to interpret for long recordings.
Files Needing Attention: apps/desktop/src/routes/recordings-overlay.tsx
Important Files Changed
| Filename |
Overview |
| apps/desktop/src/routes/editor/ExportPage.tsx |
Adds a guarded size and duration estimate beneath the export button with appropriate seconds-to-minutes formatting. |
| apps/desktop/src/routes/recordings-overlay.tsx |
Adds the estimate to thumbnail hover cards, but formats arbitrarily long durations exclusively as raw seconds. |
Prompt To Fix All With AI
### Issue 1
apps/desktop/src/routes/recordings-overlay.tsx:388
**Format long estimates readably**
The badge renders every `estimatedExportTime` value as raw seconds, so long recordings display values such as `~10800s` instead of the more readable minutes-and-seconds format used by the main export view.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.
Reviews (1): Last reviewed commit: “fix: add time and size estimation below …” | Re-trigger Greptile
Greptile also left 1 inline comment on this PR.