Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(webui): reverse log order to display newest first #875

Merged
merged 4 commits into from
Apr 22, 2024

Conversation

skrashevich
Copy link
Contributor

The applyLogStyling function in log.html has been updated to reverse the array of log lines.

The reason is because usually we need the most recent logs, not the old ones.

The applyLogStyling function in log.html has been updated to reverse the array of log lines. After parsing the JSON data, reversing the array ensures that the most recent logs appear at the top of the list. This change enhances the readability for users by displaying the logs in a descending chronological order.
Added a button to the log page allowing users to toggle the order in which logs are displayed (normal or reversed). This feature enhances user experience by providing flexibility in viewing logs. The implementation involves a boolean flag `reverseOrder` to track the current state of log order and dynamically updates the button text to reflect the current mode. Additionally, the log fetching function now conditionally reverses the log array based on this flag, ensuring that the display order matches the user's preference. This change could significantly improve usability for users needing to analyze recent events without scrolling through the entire log history.
@skrashevich skrashevich changed the title Reverse log order to display newest first feat(webui): reverse log order to display newest first Apr 18, 2024
@AlexxIT
Copy link
Owner

AlexxIT commented Apr 22, 2024

Thanks!

@AlexxIT AlexxIT merged commit 675476a into AlexxIT:master Apr 22, 2024
@AlexxIT AlexxIT added this to the v1.9.0 milestone Apr 28, 2024
@AlexxIT
Copy link
Owner

AlexxIT commented Apr 30, 2024

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.

None yet

2 participants