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

frontend: Add TimeAgo component that displays and auto refreshes time #2727

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

sniok
Copy link
Contributor

@sniok sniok commented Jan 3, 2025

Fixes #2726

This PR adds TimeAgo component that updates every second to display time since the given date.
Only the date itself updates and the table is not re-rendered so there's pretty much no impact on performance.

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jan 3, 2025
Copy link
Collaborator

@joaquimrocha joaquimrocha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. We may also be able to optimize this by understanding what is the right period to update this, based on the format + abbreviation.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jan 6, 2025
@joaquimrocha joaquimrocha merged commit d826085 into main Jan 6, 2025
18 checks passed
@joaquimrocha joaquimrocha deleted the age-refresh branch January 6, 2025 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The "Age" column does not refresh
2 participants