Conversation
|
@daniellekirkwood tagged as discussed. I think this just needs some thoughts about how best to fit it into the profile view. I'm happy to wait if existing designs are in the process of landing, I can just rebase. |
And automatically refresh the timezone every minute.
dbkr
left a comment
There was a problem hiding this comment.
Otherwise looks good! I think your disable-line needs to be on the preceding line.
| /** | ||
| * Fetch a user's delclared timezone through their profile, and return | ||
| * a friendly string of the current time for that user. This will keep | ||
| * in sync with the current time, and will be refreshed once a minute. |
There was a problem hiding this comment.
Updating feels like it might be overkill here, especially since it doesn't look like there's any UI to suggest it's dynamically refreshing, but seems fine I guess.
There was a problem hiding this comment.
It just felt weird to me that the time would slowly drift out of date if you left the user open for a minute. I suppose it's an incredibly minor thing, but also I don't think it's a costly rerender.
|
Closing in favor of element-hq/matrix-react-sdk#20 |
Implements MSC4175
Depends on matrix-org/matrix-js-sdk#4391
Design provided at https://www.figma.com/design/gpxHFDTNK796n84r1ZxYG2?node-id=1360-28954#941702635
This provides a basic presentation of a user's current time & timezone on their profile. It also adds a setting to allow publishing the time.
Timezone in user profile
Timezone publish setting
Checklist
public/exportedsymbols have accurate TSDoc documentation.