Differential D20314

Fix two minor timezone display issues
ClosedPublic
Actions

Authored by epriestley on Mar 22 2019, 10:07 PM.

Tags

None

Referenced Files

	F15527918: D20314.id48468.diff
	Tue, Apr 22, 7:30 AM

	F15520955: D20314.id48480.diff
	Sun, Apr 20, 9:35 AM

	F15515550: D20314.diff
	Fri, Apr 18, 11:25 AM

	F15513967: D20314.id48468.diff
	Thu, Apr 17, 11:24 PM

	F15511538: D20314.id48468.diff
	Thu, Apr 17, 2:50 AM

	F15507602: D20314.diff
	Tue, Apr 15, 9:37 PM

	F15477124: D20314.id48480.diff
	Mon, Apr 7, 12:41 PM

	F15436600: D20314.id48480.diff
	Tue, Mar 25, 3:25 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T13263: Plans: 2019 Week 11-15 Bonus Content

Commits

rP1d73ae3b50a6: Fix two minor timezone display issues

Summary

Ref T13263. Two minor issues:

The "reconcile" dialog shows the wrong sign because JS signs differ from normal signs (for example, PST or PDT or whatever we're in right now is shown as "UTC+7", but should be "UTC-7").
The big dropdown of possible timezones lumps "UTC+X:30" timezones into "UTC+X".

Test Plan

Reconciled "America/Nome", saw negative UTC offsets for "America/Nome" and "America/Los_Angeles" (previously: improperly positive).
Viewed the big timzone list, saw ":30" and ":45" timezones grouped/labeled more accurately.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 22 2019, 10:07 PM

Harbormaster completed remote builds in B22332: Diff 48468.Mar 22 2019, 10:09 PM

epriestley requested review of this revision.Mar 22 2019, 10:09 PM

amckinley accepted this revision.Mar 25 2019, 5:37 PM

This revision is now accepted and ready to land.Mar 25 2019, 5:37 PM

Closed by commit rP1d73ae3b50a6: Fix two minor timezone display issues (authored by epriestley). · Explain WhyMar 25 2019, 5:53 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

settings/

controller/

PhabricatorSettingsTimezoneController.php

5 lines

setting/

PhabricatorTimezoneSetting.php

13 lines

Diff 48480

src/applications/settings/controller/PhabricatorSettingsTimezoneController.php

Loading...

src/applications/settings/setting/PhabricatorTimezoneSetting.php

Loading...