-
Notifications
You must be signed in to change notification settings - Fork 46
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
Display reminder of user timezone above calendar #371
Comments
@funkycram Is this one you could look at? |
@samuk it's not exactly your request. I think we need to show the time zone beside the time, as suggested in this PR: Because events can be viewed also in stream, and on the calendar the time don't displays on the Month view. I didn't add the link to update the time zone because I think it will overload the view and it's not a common practice on Humhub to show links to the settings. E.g. on the notifications list, we don't show a link to change the settings. Perhaps with the "Tour" module we could help the user updating the time zone and other settings, because this module shows for new accounts. Later, we don't need to always see links to update the time zone or other settings. |
Thanks for the PR. Hmm, I unsure that we should display the time zone that prominent. Some ideas:
|
Thanks @luke- for these ideas!
But I think it would be a good idea for new account creation. We could show the time zone field on the registration form and JS would automatically, if available from the browser, select the browser's timezone.
|
"Later, we don't need to always see links to update the time zone or other settings." The problem is users are not static. They have a tendency to move. The browser detection on signup sounds like a nice idea and worth doing, it still doesn't solve the problem as users move timezones though. Can we do the 'show the current user time zone in calendar view' option alongside browser detection? Thanks Sam |
@Eladnarlea Can you please take a look here? |
Forcing a timezone this way does not work well on Browsers that have tracking prevention enabled like Brave. My suspicion is that most platforms open a dialog to ask users for confirmation changes due to a variety of corner-cases where real life collides with this detection mechanism. |
@Eladnarlea what do you think about it? Thanks! |
@marc-farre I will discuss this matter with @luke-. I like some suggestions but it is more complex and implementation requires more time as it might seem as it comes along with other timezone problems. At the moment we focus on wrapping up other projects but your issue is definitely on our list of things to improve. |
Thanks @Eladnarlea. @marc-farre is it possible to do some short term fix on our local instances while we wait for this to be resolved upstream? It's an ongoing issue for us. People miss meetings with some regularity. |
I've been pondering about a good solution recently and I must say I really like @marc-farre approach of adding the timezone For those that are aware of different timezones (not all users are in my experience) it makes it a full statement. When it comes to adjusting times I would vote for the server always emitting timestamps that include a timezone information (possibly the server timezone) in an HTML Javascript can now take care of adjusting the I also vote for displaying time in the timezone reported by the devices used for viewing a page. Given this is mostly an issue for people travelling and/ or living in countries with multiple timezones and my personal experience with both scenarios portable devices (phones, laptops) have become pretty good at automatically picking up my timezone changes. In my experience relying on the OS+Browser reported timezone will create the less issues than requiring users to change their timezone setting within a HumHub instance. Update [1] Yii::$app->formatter->asDateTime(time(), "php:T") but it turns out the mapping between different locations to timezone abbreviations only works partially, resulting in more |
Users don't always correctly set or update their timezones.
Could we add some text above the calendar;
Your current timezone is <UTC +1> adjust timezone
The text was updated successfully, but these errors were encountered: