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

When hovering over an avatar, do not only display local time but also calculate the time difference compared to you (similar to how GitHub does this) #48626

Open
Tracked by #43763
sorbaugh opened this issue Oct 9, 2024 · 6 comments

Comments

@sorbaugh
Copy link
Contributor

sorbaugh commented Oct 9, 2024

Subtask: #43763

@sorbaugh
Copy link
Contributor Author

sorbaugh commented Oct 9, 2024

cc @nickvergessen something that is relevant for Talk too?

@nickvergessen
Copy link
Member

Will automatically appear in Talk if modified correctly in the LocalTimeProvider

@sorbaugh
Copy link
Contributor Author

Notes:

  • Might need backend work. Judging from nickvergessen's comment this would require "only" a change in one line of code but needs to be tested.
  • We need to improve the user popup a bit. There's additional data that could be added. Consensus is generally the user popup could use some love.

@marcoambrosini marcoambrosini moved this to 🏗️ At engineering in 🖍 Design team Oct 11, 2024
@marcoambrosini marcoambrosini self-assigned this Oct 11, 2024
@sorbaugh sorbaugh moved this from 📄 To do (~10 entries) to 🏗️ In progress in 📁 Files team Dec 2, 2024
@sorbaugh
Copy link
Contributor Author

sorbaugh commented Dec 2, 2024

@marcoambrosini our assumption would be something like displaying "Local time: 9:41 AM (3 hours behind)", "Local time: 9:41 AM (2 hours ahead)"

We're not sure yet. Wdyt?

Image

@marcoambrosini
Copy link
Member

I think that works, and we should put that in a new line

@sorbaugh
Copy link
Contributor Author

sorbaugh commented Dec 2, 2024

I think that works, and we should put that in a new line

cc @yemkareems

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🏗️ In progress
Status: 🏗️ At engineering
Development

No branches or pull requests

4 participants