Page MenuHomePhabricator
Differential D12742

First stab at Calendar day view sidebar
ClosedPublic
Actions

Authored by lpriestley on May 7 2015, 1:21 AM.
Tags
None
Referenced Files
F14010581: D12742.id30617.diff
Thu, Oct 31, 10:28 AM
F14010532: D12742.id30617.diff
Thu, Oct 31, 9:39 AM
F14000590: D12742.id30619.diff
Thu, Oct 24, 11:07 PM
F13990121: D12742.diff
Tue, Oct 22, 1:22 AM
F13973127: D12742.id.diff
Oct 17 2024, 11:15 PM
F13958466: D12742.id30616.diff
Oct 14 2024, 2:56 PM
Unknown Object (File)
Oct 8 2024, 6:33 AM
Unknown Object (File)
Oct 7 2024, 2:42 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
chad
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T4393: Implement "Daily/Weekly" Calendar Mock
Commits
Restricted Diffusion Commit
rP7fd401d0e023: First stab at Calendar day view sidebar
Summary

Ref T4393, First stab at Calendar day view sidebar

Test Plan

Open Calendar day view, sidebar should show today and the next 6 days, empty or not.

Diff Detail

Repository
rP Phabricator
Branch
calendardaysidebar
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5811
Build 5830: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

lpriestley updated this revision to Diff 30616.May 7 2015, 1:21 AM
lpriestley retitled this revision from to First stab at Calendar day view sidebar.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T4393: Implement "Daily/Weekly" Calendar Mock.
Herald added reviewers: chad, Blessed Reviewers. · View Herald TranscriptMay 7 2015, 1:21 AM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
lpriestley updated this revision to Diff 30617.May 7 2015, 1:21 AM
lpriestley edited edge metadata.

Updating celerity

chad edited edge metadata.May 7 2015, 1:27 AM

This_thread_is_worthless_without_pictures_zps73b36e33.jpg (255×332 px, 19 KB)

lpriestley added a comment.May 7 2015, 1:33 AM

@chad here you go!

Screen Shot 2015-05-06 at 6.32.03 PM.png (805×1 px, 104 KB)

epriestley accepted this revision.May 7 2015, 1:36 AM
epriestley edited edge metadata.
epriestley added inline comments.
src/view/phui/calendar/PHUICalendarDayView.php
202–204

When you only have one node, you can just pass $name instead of array($name).

206–243

Same for three of these four.

263

pht()

265

pht()

This revision is now accepted and ready to land.May 7 2015, 1:36 AM
lpriestley updated this revision to Diff 30618.May 7 2015, 1:40 AM
lpriestley marked 4 inline comments as done.
lpriestley edited edge metadata.

Cleaning up phutil_tag calls and some missed pht opportunities

Closed by commit rP7fd401d0e023: First stab at Calendar day view sidebar (authored by lpriestley, committed by lpriestley). · Explain WhyMay 7 2015, 1:41 AM
This revision was automatically updated to reflect the committed changes.
chad added a comment.May 7 2015, 1:43 AM

For the sidebar, there should already be a PHUI library for that, PHUICalendarListView or something?

Revision Contents
Changeset List

PathSize
resources/
celerity/
4 lines
src/
applications/
calendar/
query/
4 lines
infrastructure/
time/
11 lines
view/
phui/
calendar/
148 lines
webroot/
rsrc/
css/
phui/
calendar/
23 lines

Diff 30618

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/calendar/query/PhabricatorCalendarEventSearchEngine.php

Loading...
View Options

src/infrastructure/time/PhabricatorTime.php

Loading...
View Options

src/view/phui/calendar/PHUICalendarDayView.php

Loading...
View Options

webroot/rsrc/css/phui/calendar/phui-calendar-day.css

Loading...
Phabricator · Built by Phacility