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

	F15423273: D12978.diff
	Sat, Mar 22, 2:25 PM

	F15416570: D12978.diff
	Thu, Mar 20, 12:11 PM

	F15380169: D12978.id31282.diff
	Fri, Mar 14, 1:46 AM

	F15377725: D12978.id31283.diff
	Thu, Mar 13, 10:27 AM

	F15363899: D12978.id31282.diff
	Tue, Mar 11, 12:53 PM

	Unknown Object (File)
	Tue, Feb 25, 9:52 AM

	Unknown Object (File)
	Feb 13 2025, 3:25 AM

	Unknown Object (File)
	Feb 8 2025, 4:49 AM

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...