-
Notifications
You must be signed in to change notification settings - Fork 128
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
[Feature request]: Nodes quick options #307
base: master
Are you sure you want to change the base?
Conversation
The image you linked 404s |
@Hunter275 it is just gif of a feature. Now it should work. |
This should be good to go if there aren't any more wishes. EDIT: |
@Hunter275 now it should be ready |
timestamp: number; | ||
} | ||
|
||
const getTimeAgo = (timestamp: number): string => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd prefer we avoid using setIntervals to manually calculate time. There is a Browser API Intl which offers a built in api to calculate elapsed time, here is an article with some code you could re-use:
https://dev.to/madsstoumann/showing-time-ago-in-a-social-feed-using-intlrelativetimeformat-5ceo
Similar features are in mobile apps.
When you are on the "Nodes" page, you can click on the node name and a popup pops out.
For now, the popup contains Traceroute and Request position buttons.