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

Adds timezone to ltt time helper #152

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

colindean
Copy link

This might help folks outside of CET get the time right.

The TZ is in the title on the link but someone might not think to look at that.

N.b. I'm making this edit in GitHub so I've not verified that it looks right.

This might help folks outside of CET get the time right.

The TZ is in the title on the link but someone might not think to look at that.
@johanvdw
Copy link
Member

johanvdw commented Feb 5, 2021

Thanks for your help.
This gets quite large, wonder if there is a nicer method.
image

@colindean
Copy link
Author

Another strategy could be to plaster everywhere that all times everywhere on the site are CET. I'm not sure how many places would have to change, though, so modifying the very function that renders the time seemed more appropriate but I will concede that it is probably overkill.

@boegel
Copy link
Contributor

boegel commented Jan 6, 2022

I strongly support this proposal.

With FOSDEM'22 being fully online again, I think it's very important to make it crystal clear that all times mentioned on the website are in Brussels time (CET).

Maybe the width of the Start and End columns should be increased slightly to avoid line-wrapping of the time + CET?

@kergon
Copy link
Member

kergon commented Jan 6, 2022

I don't want it everywhere like that. There's no shortcut to doing a change like this properly.

@boegel
Copy link
Contributor

boegel commented Jan 6, 2022

How about a line (in bold) like "All times are in CET (UTC+1) time zone" on top of the Event table (or next both to Saturday and Sunday)?

@kergon
Copy link
Member

kergon commented Jan 6, 2022

By properly I mean a complete review of the website, looking at all the places where a time is shown, categorising the different types of cases found, and then deciding how best to remove the current ambiguity - if possible, even letting the viewer specify their preferred timezone. Different pages might need different solutions - until someone's worked through it all I don't know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants