feat(logs): add Slack-like scroll UX to service, deployment and cluster logs#2592
feat(logs): add Slack-like scroll UX to service, deployment and cluster logs#2592acarranoqovery wants to merge 3 commits intonew-navigationfrom
Conversation
…ter logs - When the user scrolls up, the view sticks to the current position - A "Jump to latest log" button appears at the bottom with a badge showing the count of new log lines received while scrolled away - Scrolling back to the bottom automatically resumes live streaming - Pausing on scroll now requires a 80px threshold to avoid accidental triggers - Fixed deployment logs not rendering new lines while scrolled up (chunks were blocked from flushing when paused) - Renamed internal state: pauseLogs → isScrollPaused, newLogsAvailable → bufferedLogsCount for clarity
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## new-navigation #2592 +/- ##
=================================================
Coverage ? 45.36%
=================================================
Files ? 702
Lines ? 17014
Branches ? 5046
=================================================
Hits ? 7718
Misses ? 7899
Partials ? 1397
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@acarranoqovery Looks good to me UX-wise, much better behavior than today. UI wise, I think I would probably remove the icon as it's a bit redundant and visually a bit clunky when we have both the numbers of new logs + the icon (too visually noisy imo). If we want to keep the icon, then just switch it to a arrow-bottom, currently the icon use is the one that is traditionally used for "download" actions |

Summary
pauseLogs→isScrollPaused,newLogsAvailable(boolean) →bufferedLogsCount(number)Screenshots / Recordings
N/A
Testing
yarn testoryarn test -u(if you need to regenerate snapshots)yarn formatyarn lintPR Checklist
feat(service): add new Terraform service) - required for semantic-release