Paths

Table of Contentst

Differential D12928

Skeleton of calendar icon infrastructure
ClosedPublic
Actions

Authored by lpriestley on May 19 2015, 5:58 PM.

Tags

None

Referenced Files

	F19939317: D12928.diff
	Tue, Apr 14, 12:10 PM

	F19910698: D12928.id31118.diff
	Thu, Apr 2, 9:17 AM

	F19900928: D12928.diff
	Mon, Mar 30, 6:49 AM

	F19900925: D12928.diff
	Mon, Mar 30, 6:49 AM

	F19821851: D12928.id.diff
	Mar 7 2026, 1:17 AM

	F19815737: D12928.id.diff
	Mar 5 2026, 12:26 AM

	F19726264: D12928.id31116.diff
	Feb 13 2026, 5:48 PM

	F18876124: D12928.diff
	Nov 5 2025, 9:47 PM

Subscribers

Details

Reviewers

epriestley
chad

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T7936: Give Calendar events custom icons

Commits

Restricted Diffusion Commit
rP7bc008d0e2cc: Skeleton of calendar icon infrastructure

Summary

Ref T7936, Skeleton of calendar icon infrastructure

Test Plan

Navigate to install/calendar/icon/, observe temporary calendar icons

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 31116.May 19 2015, 5:58 PM

lpriestley retitled this revision from to Skeleton of calendar icon infrastructure.

lpriestley updated this object.

lpriestley edited the test plan for this revision. (Show Details)

lpriestley added a reviewer: epriestley.

lpriestley added a task: T7936: Give Calendar events custom icons.

Herald added reviewers: chad, Blessed Reviewers. · View Herald TranscriptMay 19 2015, 5:58 PM

Herald added subscribers: epriestley, Korvin. · View Herald Transcript

epriestley accepted this revision.May 19 2015, 6:09 PM

epriestley edited edge metadata.

epriestley added inline comments.

src/applications/calendar/controller/PhabricatorCalendarEventEditIconController.php
8–13	Prefer the slightly more modern/simpler: `handleRequest(AphrontRequest $request)` method (instead of `processRequest()`) then `$id = $request->getURIData('id');`, and pull `$viewer` with `$this->getViewer()`.
30	Trailing slash probably incorrect?

This revision is now accepted and ready to land.May 19 2015, 6:10 PM

retro-garbage

Closed by commit rP7bc008d0e2cc: Skeleton of calendar icon infrastructure (authored by lpriestley, committed by lpriestley). · Explain WhyMay 19 2015, 6:13 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

2 lines

src/

__phutil_library_map__.php

4 lines

applications/

calendar/

application/

PhabricatorCalendarApplication.php

4 lines

controller/

PhabricatorCalendarEventEditIconController.php

102 lines

icon/

PhabricatorCalendarIcon.php

49 lines

webroot/

rsrc/

css/

application/

calendar/

calendar-icon.css

28 lines

Diff 31119

resources/celerity/map.php

Loading...

src/__phutil_library_map__.php

Loading...

src/applications/calendar/application/PhabricatorCalendarApplication.php

Loading...

src/applications/calendar/controller/PhabricatorCalendarEventEditIconController.php

Loading...

src/applications/calendar/icon/PhabricatorCalendarIcon.php

Loading...

webroot/rsrc/css/application/calendar/calendar-icon.css

Loading...