feat(logs-viewer): add toggleable log ordering and update scroll-to-l… #342

Closed
opened 2026-04-05 16:19:25 +02:00 by MrUnknownDE · 0 comments
Owner

Originally created by @simlarsen on 10/23/2025

…atest behavior

  • add isDescending state and useMemo displayLogs to reverse logs when needed
  • rename showScrollToBottom -> showScrollToLatest and replace scrollToBottom with scrollToLatest
  • convert handleScroll to useCallback and make it order-aware (top vs bottom)
  • update scroll button icon/title and ensure scroll position recalculates after toggling order
  • render and count displayLogs (use reversed list) and update empty state checks
  • minor comment update in Dashboard LogsViewer

Title of this pull request?

Small Description?

Pull Request Checklist:

  • Please make sure all jobs pass before requesting a review.
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).
  • Have you lint your code locally before submission?
  • Did you write tests where appropriate?

Screenshots (if appropriate):

*Originally created by @simlarsen on 10/23/2025* …atest behavior - add isDescending state and useMemo displayLogs to reverse logs when needed - rename showScrollToBottom -> showScrollToLatest and replace scrollToBottom with scrollToLatest - convert handleScroll to useCallback and make it order-aware (top vs bottom) - update scroll button icon/title and ensure scroll position recalculates after toggling order - render and count displayLogs (use reversed list) and update empty state checks - minor comment update in Dashboard LogsViewer ### Title of this pull request? ### Small Description? ### Pull Request Checklist: - [ ] Please make sure all jobs pass before requesting a review. - [ ] Put `closes #XXXX` in your comment to auto-close the issue that your PR fixes (if such). - [ ] Have you lint your code locally before submission? - [ ] Did you write tests where appropriate? ### Related Issue? ### Screenshots (if appropriate):
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github/oneuptime#342