Skip to content

Improve display of Webauthn keys' creation time #28262

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

Closed

Conversation

earl-warren
Copy link
Contributor

  • Unify how the creation time of webauthn keys are shown with GPG and SSH keys.
  • Instead of using the time since, show the date that the key was created.

Refs: https://codeberg.org/forgejo/forgejo/pulls/1794

(cherry picked from commit 0a2db6215cb10a6bfcb558f5afabc7fba208e3f8)

- Unify how the creation time of webauthn keys are shown with GPG and
SSH keys.
- Instead of using the time since, show the date that the key was
created.

Refs: https://codeberg.org/forgejo/forgejo/pulls/1794

(cherry picked from commit 0a2db6215cb10a6bfcb558f5afabc7fba208e3f8)
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Nov 28, 2023
@github-actions github-actions bot added the modifies/templates This PR modifies the template files label Nov 28, 2023
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Nov 28, 2023
@wxiaoguang
Copy link
Contributor

Is it really good to do so?

Before: end users could know the hour/minute/second when hover on the "time"

Now: end users could only know the "year/month/day" , but they wouldn't be able to know the detailed time.

KN4CK3R pushed a commit that referenced this pull request Jan 2, 2024
There was a question about "how to improve the datetime display for
SSH/PGP/WebAuthn"
#28262 (comment)

The root problem is that `DateTime` misses the "data-tooltip-content"
attribute, which should be used to make the tooltip popup smoothly.

Now the UI is consistent and the end users could see the detailed
hour/minute/second easily by hovering the element.


![image](https://github.com/go-gitea/gitea/assets/2114189/2211336f-d59d-4f64-a83b-099f8ef6d29b)


![image](https://github.com/go-gitea/gitea/assets/2114189/f02a9c86-476d-48d6-aece-85a800235fbd)
@wxiaoguang
Copy link
Contributor

No response for long time.

The question has been answered by Make template DateTime show proper tooltip #28677 .

The root problem is that some attributes are missing for relative-time.

@wxiaoguang wxiaoguang closed this Jan 7, 2024
fuxiaohei pushed a commit to fuxiaohei/gitea that referenced this pull request Jan 17, 2024
There was a question about "how to improve the datetime display for
SSH/PGP/WebAuthn"
go-gitea#28262 (comment)

The root problem is that `DateTime` misses the "data-tooltip-content"
attribute, which should be used to make the tooltip popup smoothly.

Now the UI is consistent and the end users could see the detailed
hour/minute/second easily by hovering the element.


![image](https://github.com/go-gitea/gitea/assets/2114189/2211336f-d59d-4f64-a83b-099f8ef6d29b)


![image](https://github.com/go-gitea/gitea/assets/2114189/f02a9c86-476d-48d6-aece-85a800235fbd)
silverwind pushed a commit to silverwind/gitea that referenced this pull request Feb 20, 2024
There was a question about "how to improve the datetime display for
SSH/PGP/WebAuthn"
go-gitea#28262 (comment)

The root problem is that `DateTime` misses the "data-tooltip-content"
attribute, which should be used to make the tooltip popup smoothly.

Now the UI is consistent and the end users could see the detailed
hour/minute/second easily by hovering the element.


![image](https://github.com/go-gitea/gitea/assets/2114189/2211336f-d59d-4f64-a83b-099f8ef6d29b)


![image](https://github.com/go-gitea/gitea/assets/2114189/f02a9c86-476d-48d6-aece-85a800235fbd)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 1 This PR needs approval from one additional maintainer to be merged. modifies/templates This PR modifies the template files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants