#4259 time zone availability#4276
Conversation
wavehassman
left a comment
wavehassman
left a comment
There was a problem hiding this comment.
I want to make it clear that the times are shown in your local time zone on the calendar and on the availability page. Maybe just a comment like "Times shown in your local timezone (PST)" so no one is confused
| <Box display="flex" justifyContent="space-between" mb={1}> | ||
| <Typography variant="subtitle1">Available times in green</Typography> | ||
| <Typography variant="subtitle1"> | ||
| Available times in <span style={{ color: HeatmapColors[3] }}>green</span>. All times are in local time,{' '} |
There was a problem hiding this comment.
when I invert availability, this doesn't change
also unecessary comment, we just need the timezone
There was a problem hiding this comment.
could you explain a bit more? Why does it need to change when inverting availability? The time zone shouldn't change when inverting availability as it is unrelated to time zones.
For the second line, I am guessing that you are referring to changing the color "green" to green. This was just an additional extra touch, which while unnecessary, does provide consistency with the messages about what underlining and red names mean when hovering over a time
There was a problem hiding this comment.
You can invert availability, so available times are in white. This doesn't change with that and stays green
2 participants
Changes
The availability calendar changes depending on user's timezone.
To Do
Any remaining things that need to get done
Checklist
It can be helpful to check the
ChecksandFiles changedtabs.Please review the contributor guide and reach out to your Tech Lead if anything is unclear.
Please request reviewers and ping on slack only after you've gone through this whole checklist.
yarn.lockchanges (unless dependencies have changed)Closes #4259