Page MenuHomePhabricator
Differential D16956

Make the modal choice between "Edit This Event" and "Edit Future Events" more clear
ClosedPublic
Actions

Authored by epriestley on Nov 28 2016, 5:49 PM.
Tags
None
Referenced Files
F13060392: D16956.diff
Fri, Apr 19, 5:46 PM
Unknown Object (File)
Fri, Apr 12, 6:34 AM
Unknown Object (File)
Fri, Apr 12, 6:34 AM
Unknown Object (File)
Fri, Apr 12, 6:30 AM
Unknown Object (File)
Fri, Apr 12, 6:07 AM
Unknown Object (File)
Fri, Apr 12, 3:17 AM
Unknown Object (File)
Sun, Mar 31, 5:55 AM
Unknown Object (File)
Sun, Mar 31, 5:55 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T11816: Calendar: Feedback and Suggestions
T11909: Recurring Events: "Edit all Future Events" edits past instances as well
Commits
rP5cc4f6407ceb: Make the modal choice between "Edit This Event" and "Edit Future Events" more…
Summary

Fixes T11909. Ref T11816. Instead of offering a dropdown with choices between "Edit/Cancel/Reinstate This Event" and "Edit/Cancel/Reinstate Future Events", make the choice more explicit.

This dialog ends up pretty wordy but this edit is rare, so I think that's alright.

Test Plan

Screen Shot 2016-11-28 at 9.44.05 AM.png (1×1 px, 191 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 40807.Nov 28 2016, 5:49 PM
epriestley retitled this revision from to Make the modal choice between "Edit This Event" and "Edit Future Events" more clear.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added tasks: T11909: Recurring Events: "Edit all Future Events" edits past instances as well, T11816: Calendar: Feedback and Suggestions.
epriestley mentioned this in T11909: Recurring Events: "Edit all Future Events" edits past instances as well.Nov 28 2016, 5:50 PM
chad accepted this revision.Nov 28 2016, 6:46 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 28 2016, 6:46 PM
Closed by commit rP5cc4f6407ceb: Make the modal choice between "Edit This Event" and "Edit Future Events" more… (authored by epriestley, committed by epriestley). · Explain WhyNov 28 2016, 6:48 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
calendar/
controller/
52 lines
32 lines

Diff 40809

View Options

src/applications/calendar/controller/PhabricatorCalendarEventCancelController.php

Loading...
View Options

src/applications/calendar/controller/PhabricatorCalendarEventEditController.php

Loading...
Phabricator · Built by Phacility