Page MenuHomePhabricator

Clean up recurring event information on Calendar events
ClosedPublic

Authored by epriestley on Jul 28 2016, 2:55 PM.
Tags
None
Referenced Files
F13059987: D16339.diff
Fri, Apr 19, 5:08 PM
Unknown Object (File)
Thu, Apr 18, 8:25 AM
Unknown Object (File)
Tue, Apr 16, 5:39 AM
Unknown Object (File)
Sun, Apr 14, 5:02 PM
Unknown Object (File)
Thu, Apr 11, 7:42 AM
Unknown Object (File)
Sat, Apr 6, 1:14 PM
Unknown Object (File)
Fri, Mar 29, 9:01 AM
Unknown Object (File)
Fri, Mar 29, 12:12 AM
Subscribers
None

Details

Summary

Ref T11326. This adds prev/next links for recurring events (ala D16179) and moves the "accept/decline" buttons closer to the invite list. This might need some fiddling, but should be a little more human-friendly.

Test Plan

Screen Shot 2016-07-28 at 7.54.26 AM.png (1×1 px, 153 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Clean up recurring event information on Calendar events.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.

Maybe move [ Accept ] [ Decline ] into the main header?

This revision is now accepted and ready to land.Jul 28 2016, 4:11 PM

I moved them out of the main header in this diff (putting them closer to the invite list) since they felt kind of weird to me up there, but I can put them back. I'm not really sure we need them.

Is this an event you've been invited to? Or just an event you happen to see? I expect if I'm invited, the actions are more prominent. If it it's just some random event, then maybe just a join button? I haven't played with these much yet.

The big action buttons only show up if you have an unresponded invite.

There are normal ways to interact with invite status in the standard action list (e.g., "Decline Event") to an event you previously marked yourself attending.

Yeah sorry, I slightly prefer them in the header now that I've had some caffeine. :(

Haha, no problem. I expect a lot of Calendar stuff to need a few more UI iterations, I'm just trying to move things somewhere in the right direction and get the backend kind of reasonably well organized for now.

The buttons feel weird in this diff to me too, just also weird in the other spot. ¯\_(ツ)_/¯

epriestley edited edge metadata.
  • Put the buttons back in the header.
This revision was automatically updated to reflect the committed changes.