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

	F19061274: D12786.id.diff
	Nov 29 2025, 11:57 AM

	F19054312: D12786.diff
	Nov 28 2025, 12:04 PM

	F18852348: D12786.id30735.diff
	Oct 31 2025, 8:05 AM

	F18831311: D12786.id.diff
	Oct 25 2025, 12:15 PM

	F18827755: D12786.diff
	Oct 24 2025, 11:59 AM

	F18822920: D12786.id30737.diff
	Oct 23 2025, 9:18 AM

	F18770879: D12786.diff
	Oct 8 2025, 3:52 PM

	F18751023: D12786.id30734.diff
	Oct 4 2025, 9:33 AM

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