Paths

Table of Contentst

Differential D12786

Calendar day view should notify user if they have navigated away from the query range dates.
ClosedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Apr 25, 1:59 AM

	Unknown Object (File)
	Sat, Apr 20, 4:26 AM

	Unknown Object (File)
	Sat, Apr 20, 4:26 AM

	Unknown Object (File)
	Sat, Apr 20, 4:26 AM

	Unknown Object (File)
	Sat, Apr 20, 4:26 AM

	Unknown Object (File)
	Sat, Apr 20, 4:19 AM

	Unknown Object (File)
	Sat, Apr 20, 3:40 AM

	Unknown Object (File)
	Tue, Apr 9, 2:38 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T8104: Viewer should be notified if viewed "day view" or "month view" is out of the selected query range.

Commits

Restricted Diffusion Commit
rPb67ad3c4e06a: Calendar day view should notify user if they have navigated away from the…

Summary

Ref T8104, Calendar day view should notify user if they have navigated away from the query range dates.

Test Plan

Open Calendar day view, choose May 12-13 range, execute query, page through to May 11, day view should show error that day is out of range.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 30734.May 9 2015, 8:13 PM

lpriestley retitled this revision from to Calendar day view should notify user if they have navigated away from the query range dates..

lpriestley updated this object.

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

lpriestley added a reviewer: epriestley.

lpriestley added a task: T8104: Viewer should be notified if viewed "day view" or "month view" is out of the selected query range. .

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

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

Adding "part of day is out of range" errors

epriestley accepted this revision.May 9 2015, 8:52 PM

epriestley edited edge metadata.

epriestley added inline comments.

src/view/phui/calendar/PHUICalendarDayView.php
346–348	I think you can just: $header_text = $display_day->format('l, F j, y'); The `clone` shouldn't be necessary because `format()` doesn't modify the object.
434–435	Does this still have callers?

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

date format cleanup

Closed by commit rPb67ad3c4e06a: Calendar day view should notify user if they have navigated away from the… (authored by lpriestley, committed by lpriestley). · Explain WhyMay 9 2015, 9:19 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

calendar/

query/

PhabricatorCalendarEventSearchEngine.php

2 lines

view/

phui/

calendar/

PHUICalendarDayView.php

50 lines

Diff 30737

src/applications/calendar/query/PhabricatorCalendarEventSearchEngine.php

Loading...

src/view/phui/calendar/PHUICalendarDayView.php

Loading...