Page MenuHomePhabricator

Fixes T4815
Needs ReviewPublic

Authored by fejnartal on Apr 20 2017, 9:58 PM.
Tags
None
Referenced Files
F18770249: D17749.id.diff
Wed, Oct 8, 11:33 AM
F18748652: D17749.diff
Fri, Oct 3, 11:51 PM
F18748408: D17749.diff
Fri, Oct 3, 9:31 PM
F18659768: D17749.diff
Sep 23 2025, 3:36 PM
F18639469: D17749.diff
Sep 17 2025, 1:59 PM
F18621248: D17749.diff
Sep 15 2025, 7:29 AM
F18031499: D17749.id.diff
Aug 3 2025, 12:02 AM
F17830046: D17749.id.diff
Jul 26 2025, 10:38 AM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

Replicated the class PHUIPagerView.php to implement a [Prev][Next] pager
with fixed pageSize=1 and surroundingPages=0.

I had to work around the fact that in Phriction the last version of a Document
can also be referenced as v=0.

This caused the pager to diplay the [Next] button with a link to version=1
when viewing the last version of a wiki Document.

Future improvements:
Currently, the first version of a Document displays the [Prev] button with a link
to the last version. I haven't found any elegant workaround to this behaviour, so I
have left it untouched.

This also causes wiki Documents with only 1 version to display the [Prev] button linked
to itself.

Test Plan

Visit any wiki page with more than 1 versions.
Navigate backward: always using the [Prev] button.
Navigate forward: always using the [Next] button.
Visit a wiki page with only 1 version.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16621
Build 22163: arc lint + arc unit