Differential D11963

Show "Show Older" for transactions the user has not interacted with
ClosedPublic
Actions

Authored by epriestley on Mar 4 2015, 12:19 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Dec 6, 1:32 PM

	Unknown Object (File)
	Thu, Dec 5, 2:26 PM

	Unknown Object (File)
	Fri, Nov 29, 10:30 AM

	Unknown Object (File)
	Fri, Nov 29, 10:30 AM

	Unknown Object (File)
	Fri, Nov 29, 10:21 AM

	Unknown Object (File)
	Fri, Nov 29, 9:51 AM

	Unknown Object (File)
	Thu, Nov 28, 7:39 PM

	Unknown Object (File)
	Wed, Nov 27, 3:25 PM

Subscribers

Details

Reviewers

Maniphest Tasks

T7454: Longer Maniphest tasks not fully displayed (as Phabricator defines a result set limit as 100 rows)

Commits

Restricted Diffusion Commit
rPb09168e689ff: Show "Show Older" for transactions the user has not interacted with

Summary

Fixes T7454. We weren't triggering "Show Older" unless there were hidden events because of a previous interaction.

Test Plan

Set page size to 3.
Viewed a task with ~10 transactions.
Before patch:
- Only 3 most recent transactions visible, no way to see older ones.
- Saw "show older" appear, paged backward through transaction histroy.
Also, interacted with task and then viewed it, made sure "show older" still works.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 28804.Mar 4 2015, 12:19 PM

epriestley retitled this revision from to Show "Show Older" for transactions the user has not interacted with.

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

epriestley added a task: T7454: Longer Maniphest tasks not fully displayed (as Phabricator defines a result set limit as 100 rows).

Herald added a subscriber: epriestley. · View Herald TranscriptMar 4 2015, 12:19 PM

btrahan accepted this revision.Mar 4 2015, 6:25 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Mar 4 2015, 6:25 PM

Closed by commit rPb09168e689ff: Show "Show Older" for transactions the user has not interacted with (authored by epriestley, committed by epriestley). · Explain WhyMar 4 2015, 6:36 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

view/

control/

AphrontCursorPagerView.php

4 lines

phui/

PHUITimelineView.php

21 lines

Diff 28812

src/view/control/AphrontCursorPagerView.php

Loading...

src/view/phui/PHUITimelineView.php

Loading...