secondlife / viewer

🖥️ Second Life's official client
GNU Lesser General Public License v2.1
206 stars 53 forks source link

AM/PM selector in new scheduled region restart floater is confusing #2172

Open canny[bot] opened 1 month ago

canny[bot] commented 1 month ago

A new scheduled region restart floater as been added (see https://github.com/secondlife/viewer/pull/2093 also for a screenshot). However, as a user living in a country that is using a 24 hour clock, a AM/PM selector is completely unnecessary and possibly even confusing and irritating to the user if he doesn't know what this means. This is bad design and should be changed and adhere to common internationalization options.

https://secondlife.canny.io/admin/board/feature-requests/p/am-pm-selector-in-new-scheduled-region-restart-floater-is-confusing

canny[bot] commented 1 month ago

This issue has been linked to a Canny post: AM/PM selector in new scheduled region restart floater is confusing :tada:

akleshchev commented 1 month ago

Locale detection? Or a setting in preferences? If we are removing am/pm in any form, we should do the same in top right corner. And the opposite - show it everywhere if 'show am/pm' is set to be shown.

vir-linden commented 1 month ago

Probably there should be one viewer preference that determines 12 vs 24 hour, and everything else in the UI should respect that preference. Ideally we would pick the right default value for 12 vs 24 based on locale, but as long as people can switch the value to something they like that would be reasonable.

akleshchev commented 1 month ago

After discussion with steeltoe: Options should follow google's example: 1:00 AM and 13:00 image

akleshchev commented 1 month ago

These show AM/PM only and need to be updated:

Show 24 only:

Potentially other palces also use time, search for everything that uses datetime substitution