Skip to content

Conversation

c960657
Copy link
Contributor

@c960657 c960657 commented Sep 7, 2024

If the user hasn't selected a time zone, the time zone dropdown is just blank.

image

If no time is selected, the system default time zone (UTC) is used, but the user isn't told which zone this is. Let's make the default zone pre-selected.

@mjankowski
Copy link
Contributor

This would have the effect of opting a user in to whatever the default is (as opposed to keeping the blank value and inheriting from the default) at that moment in time, which would make it so that if the default ever changed the user would not see that change. Is that the intention here?

Or maybe more specifically - is the lack of visibility into what the default is the issue? Could we display that somewhere if so?

@c960657
Copy link
Contributor Author

c960657 commented Sep 9, 2024

I think it is fine to lock the time zone, just as the locale and theme are locked.

If the default time zone is changed in the future for whatever reason, it is unlikely that all the users want to have their effective time zone changed as well. You usually only want to change your time zone when you are (semi-)permanently moving to a different country/state.

@mjankowski
Copy link
Contributor

Sorry, to clarify a little bit -- if the TZ selection defaulted to the default TZ setting, it would wind up being set for any user who submitted the form, regardless of whether they intended to do so or not (ie, they came to the form to change their language pref or something unrelated to TZ). If in the future the default changed, they'd be on the old default from having submitted the form. This might be totally fine! -- just wanted to highlight.

@c960657
Copy link
Contributor Author

c960657 commented Sep 10, 2024

I get your point :-)

I think it's fine. I don't think users want/need the option to “follow the instance default time zone, even when it changes”.

In fact, we might even populate User#time_zone already at account creation, just like we do with the theme, so the time zone doesn't change unexpectedly at some later time. What do you think of that?

@mjankowski
Copy link
Contributor

I don't have a strong opinion on the "set a default at creation time" vs "leave blank and always inherit current default" aspect.

I wonder if this change was just changing the selected/default label (but not submitting a value) if it'd be more mergeable, and we could leave that defaults/inherit question for later.

@Gargron Gargron added this pull request to the merge queue Nov 22, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 22, 2024
@Gargron Gargron added this pull request to the merge queue Nov 22, 2024
Merged via the queue into mastodon:main with commit 2e66dd0 Nov 22, 2024
27 checks passed
@c960657 c960657 deleted the show-time-zone branch November 22, 2024 19:44
noellabo pushed a commit to fedibird/mastodon that referenced this pull request Jan 25, 2025
noellabo added a commit to fedibird/mastodon that referenced this pull request Jan 25, 2025
noellabo pushed a commit to fedibird/mastodon that referenced this pull request Jan 25, 2025
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.

3 participants