Page MenuHomePhabricator

Render timezone names more readably, with spaces rather than underscores ("America/Los Angeles", not "America/Los_Angeles").
ClosedPublic

Authored by epriestley on May 30 2019, 3:07 PM.
Tags
None
Referenced Files
F14816286: D20559.id49045.diff
Mon, Jan 27, 10:56 PM
Unknown Object (File)
Sun, Jan 26, 4:48 AM
Unknown Object (File)
Sat, Jan 25, 5:09 AM
Unknown Object (File)
Sat, Jan 25, 5:09 AM
Unknown Object (File)
Sat, Jan 25, 5:09 AM
Unknown Object (File)
Sat, Jan 25, 5:09 AM
Unknown Object (File)
Sun, Dec 29, 11:33 PM
Unknown Object (File)
Dec 16 2024, 4:50 AM
Subscribers
None

Details

Summary

See downstream https://phabricator.wikimedia.org/T902. Currently, timezones are rendered with their raw internal names (like America/Los_Angeles) which include underscores.

Replacing underscores with spaces is a more human-readable (and perhaps meaningfully better for things like screen readers, although this is pure speculation).

There's some vague argument against this, like "administrators may need to set a raw internal value in phabricator.timezone and this could mislead them", but we already give a pretty good error message if you do this and could improve hinting if necessary.

Test Plan

Viewed timezone list in Settings and the timezone "reconcile" dialog, saw a more-readable "Los Angeles".

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable