Differential D12978

First step towards a Javelin behavior for Calendar day view
ClosedPublic
Actions

Authored by lpriestley on May 22 2015, 8:13 PM.

Tags

None

Referenced Files

	F18198176: D12978.id31282.diff
	Sun, Aug 17, 11:04 PM

	F18105532: D12978.id31286.diff
	Sun, Aug 10, 6:33 PM

	F18090164: D12978.id31283.diff
	Wed, Aug 6, 2:34 PM

	F17935582: D12978.id.diff
	Wed, Jul 30, 11:26 PM

	F17907382: D12978.diff
	Tue, Jul 29, 1:16 PM

	F17859686: D12978.diff
	Sun, Jul 27, 8:59 PM

	Unknown Object (File)
	Jun 21 2025, 4:36 PM

	Unknown Object (File)
	Jun 18 2025, 10:18 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T8300: Day view should implement a a Javelin behavior

Commits

Restricted Diffusion Commit
rP8166f25c0d07: First step towards a Javelin behavior for Calendar day view

Summary

Ref T8300, First step towards a Javelin behavior for Calendar day view

Test Plan

No user facing changes.

Diff Detail

Repository

Branch

calendardayviewjsport

Lint

Lint Errors

Severity	Location	Code	Message
Error	src/view/phui/calendar/PHUICalendarDayView.php:111	XHP5	Use of Undeclared Variable
Warning	webroot/rsrc/js/application/calendar/behavior-day-view.js:1	JAVELIN5	`javelinsymbols` Not In Path
Warning	webroot/rsrc/js/application/calendar/behavior-day-view.js:7	W098	JSHintW098
Warning	webroot/rsrc/js/application/calendar/behavior-day-view.js:10	W098	JSHintW098
Warning	webroot/rsrc/js/application/calendar/behavior-day-view.js:12	W003	JSHintW003
Warning	webroot/rsrc/js/application/calendar/behavior-day-view.js:31	W014	JSHintW014
Warning	webroot/rsrc/js/application/calendar/behavior-day-view.js:42	W117	JSHintW117
Warning	webroot/rsrc/js/application/calendar/behavior-day-view.js:43	W117	JSHintW117
Warning	webroot/rsrc/js/application/calendar/behavior-day-view.js:44	W117	JSHintW117

Unit

Tests Passed

Build Status

Buildable 6217
Build 6238: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

lpriestley updated this revision to Diff 31282.May 22 2015, 8:13 PM

lpriestley retitled this revision from to First step towards a Javelin behavior for Calendar day view.

lpriestley updated this object.

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

lpriestley added a reviewer: epriestley.

lpriestley added a task: T8300: Day view should implement a a Javelin behavior.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 22 2015, 8:13 PM

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

Some lint concerns addressed

This revision is now accepted and ready to land.May 22 2015, 8:28 PM

Closed by commit rP8166f25c0d07: First step towards a Javelin behavior for Calendar day view (authored by lpriestley, committed by lpriestley). · Explain WhyMay 22 2015, 9:30 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

6 lines

src/

view/

phui/

calendar/

PHUICalendarDayView.php

24 lines

webroot/

rsrc/

js/

application/

calendar/

behavior-day-view.js

50 lines

	behavior-event-all-day.js
	event-all-day.js

16 lines

event-all-day.js

Diff 31282

resources/celerity/map.php

Loading...

src/view/phui/calendar/PHUICalendarDayView.php

Loading...

webroot/rsrc/js/application/calendar/behavior-day-view.js

Loading...

webroot/rsrc/js/application/calendar/behavior-event-all-day.js

Loading...

webroot/rsrc/js/application/calendar/event-all-day.js

Loading...