Page MenuHomePhabricator

Fix start in fancy datepicker calendar
ClosedPublic

Authored by 0 on Nov 5 2015, 7:20 AM.
Tags
None
Referenced Files
F18203348: D14409.id34819.diff
Mon, Aug 18, 11:51 AM
F18095893: D14409.id34814.diff
Fri, Aug 8, 3:08 AM
F18092302: D14409.diff
Thu, Aug 7, 9:31 AM
F18085287: D14409.diff
Tue, Aug 5, 7:07 PM
F17940524: D14409.id34819.diff
Thu, Jul 31, 4:30 AM
F17899272: D14409.id34814.diff
Jul 29 2025, 8:35 AM
F17703998: D14409.diff
Jul 16 2025, 2:55 AM
Unknown Object (File)
Jun 3 2025, 3:02 PM
Subscribers

Details

Summary

Fixes T9675.
Fixes the calculation for the difference between the first day of the month and the first day of the week. It was previously possible for this to be negative, which caused the subsequent loop to hang the browser.

Test Plan
  • Set week to start on Sunday. Observed that the calendar still renders fine.
  • Set week to start on Monday. Observed that the browser no longer hangs on November 2015.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

0 retitled this revision from to Fix start in fancy datepicker calendar.
0 updated this object.
0 edited the test plan for this revision. (Show Details)
0 added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Nov 5 2015, 3:07 PM
This revision was automatically updated to reflect the committed changes.