HomePhabricator

Make the "All Day Event" control use a checkbox instead of a dropdown

Description

Make the "All Day Event" control use a checkbox instead of a dropdown

Summary:
This feels a little cleaner:

  • Clean up transaction log a bit.
  • Use a checkbox instead of a two-option dropdown.

This is a little messy because the browser doesn't send anything if the user submits a form with an un-clicked checkbox.

We now send a dummy value ("Hey, there's definitely a checkbox in this form!") so the server can figure out what to do.

Test Plan:

  • Edited all-dayness of an event.
  • Viewed transaction log.

Reviewers: chad

Reviewed By: chad

Differential Revision: https://secure.phabricator.com/D16776

Details

Provenance
epriestleyAuthored on Oct 31 2016, 6:38 PM
epriestleyPushed on Oct 31 2016, 9:19 PM
Reviewer
chad
Differential Revision
D16776: Make the "All Day Event" control use a checkbox instead of a dropdown
Parents
rP182611ef7e57: Schedule monthly events on the 29th, 30th or 31st relative to the end of the…
Branches
Unknown
Tags
Unknown
Build Status
Buildable 14302
Build 18610: Run Core Tests