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
F14360025: D16339.id39293.diff
Fri, Dec 20, 9:53 AM
Unknown Object (File)
Thu, Dec 19, 9:54 AM
Unknown Object (File)
Tue, Dec 17, 7:59 AM
Unknown Object (File)
Tue, Dec 10, 4:38 PM
Unknown Object (File)
Mon, Dec 9, 8:47 PM
Unknown Object (File)
Wed, Dec 4, 11:11 PM
Unknown Object (File)
Mon, Nov 25, 3:49 PM
Unknown Object (File)
Nov 20 2024, 9:27 PM
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
Branch
calx1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13172
Build 16862: Run Core Tests
Build 16861: arc lint + arc unit

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.