Page MenuHomePhabricator

When scrolling forward a month in calendar date picker from 1/31, next chosen date should be 2/29, not 3/1.
ClosedPublic

Authored by lpriestley on Apr 15 2016, 8:54 PM.
Tags
None
Referenced Files
F15472426: D15727.id37994.diff
Sat, Apr 5, 1:27 PM
F15463336: D15727.id37995.diff
Tue, Apr 1, 10:33 PM
F15461765: D15727.id37994.diff
Tue, Apr 1, 11:07 AM
F15461696: D15727.id37898.diff
Tue, Apr 1, 10:25 AM
F15460372: D15727.id.diff
Mon, Mar 31, 10:23 PM
F15458626: D15727.id37899.diff
Mon, Mar 31, 5:29 AM
F15458118: D15727.diff
Sun, Mar 30, 8:57 PM
F15445090: D15727.id37899.diff
Thu, Mar 27, 12:25 PM
Subscribers

Details

Summary

Fixes T9295

Test Plan

Create event, open datepicker for start date, choose 1/31/2016, open datepicker again, click right button to scroll month. New suggested date should be 2/29/2016

Diff Detail

Repository
rP Phabricator
Branch
datepickermonthjump
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 11767
Build 14747: Run Core Tests
Build 14746: arc lint + arc unit

Unit TestsFailed

TimeTest
175 msPhabricatorCelerityTestCase::Unknown Unit Message ("")
Assertion failed, expected 'true' (at PhabricatorCelerityTestCase.php:32): When this test fails, it means the Celerity resource map is out of date. Run `bin/celerity map` to rebuild it. ACTUAL VALUE
0 msAlmanacNamesTestCase::Unknown Unit Message ("")
30 assertions passed.
0 msAlmanacServiceTypeTestCase::Unknown Unit Message ("")
1 assertion passed.
0 msAphrontHTTPSinkTestCase::Unknown Unit Message ("")
3 assertions passed.
0 msAphrontHTTPSinkTestCase::Unknown Unit Message ("")
2 assertions passed.
View Full Test Results (1 Failed · 314 Passed)

Event Timeline

lpriestley retitled this revision from to When scrolling forward a month in calendar date picker from 1/31, next chosen date should be 2/29, not 3/1..
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added reviewers: epriestley, chad.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 15 2016, 9:02 PM
This revision was automatically updated to reflect the committed changes.