Page MenuHomePhabricator

Add week numbers to Calendar month view.
ClosedPublic

Authored by lpriestley on May 17 2015, 5:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 13, 2:38 AM
Unknown Object (File)
Mar 15 2024, 4:45 AM
Unknown Object (File)
Mar 10 2024, 1:04 PM
Unknown Object (File)
Feb 14 2024, 5:39 AM
Unknown Object (File)
Feb 12 2024, 1:49 PM
Unknown Object (File)
Dec 31 2023, 8:10 PM
Unknown Object (File)
Dec 25 2023, 1:30 AM
Unknown Object (File)
Dec 22 2023, 11:18 PM
Subscribers

Details

Summary

Closes T8184, Add week numbers to Calendar month view.

Test Plan

Calendar month view should now show week numbers, incrementing on Mondays, regardless of week start day.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lpriestley retitled this revision from to Add week numbers to Calendar month view..
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added reviewers: epriestley, chad.

Screen Shot 2015-05-17 at 10.01.43 AM.png (728×836 px, 145 KB)

It's a bit cluttered on my test instance, but the week numbers are there. It looks particularly nice on weeks that start with a Monday :3

Random thought: I wonder if we need the start times at all on this view. My main thought is when scheduling or reviewing at a Month level, individual start times are not significantly important. If we had Hovercards for these events, that data could be accessed there.

epriestley edited edge metadata.
This revision is now accepted and ready to land.May 17 2015, 5:33 PM
In D12886#132896, @chad wrote:

Random thought: I wonder if we need the start times at all on this view. My main thought is when scheduling or reviewing at a Month level, individual start times are not significantly important. If we had Hovercards for these events, that data could be accessed there.

I'm the driver for this -- right now, I only have a handful of appointments so I often use the month view for everything, and the times are really useful to me. Particularly, I was using nasty workarounds in iCal to get them into titles -- if you name an event "Meeting 9PM", iCal eats the "9PM" part, so I was literally calling things "coffee at one past noon" or multi-editing the names to add with symbols to get them to work:

Screen Shot 2015-05-17 at 11.26.11 AM.png (130×296 px, 9 KB)

It turns out that the times show up in iCal if you make the window a little bigger:

Screen Shot 2015-05-17 at 11.27.21 AM.png (129×353 px, 10 KB)

This was the biggest quality-of-life Calendring improvement for me in all time.

Also I guess iCal is called "Calendar" now.

Generally, I usually have something like 4-8 total calendar events per month.

As implemented, these have some glitches (T8187), could probably be shortened (e.g., "9 AM" or "9a" instead of "9:00 AM"), might not end up really being relevant if the day view turns out to be more useful, etc., but I want to keep them around for the moment since they let me switch from iCal to Phabricator without missing anything I rely on.

Actually, your workaround to me is a slightly better design over the current UI, maybe we can do that instead? So something like:

$display = "{$title} ({$time_start})";

With the start time in grey?

This revision was automatically updated to reflect the committed changes.