Differential D10775

Make loading large changesets slightly easier
ClosedPublic
Actions

Authored by chad on Nov 3 2014, 12:01 AM.

Tags

None

Referenced Files

	F15429380: D10775.id25865.diff
	Mon, Mar 24, 1:52 AM

	F15420694: D10775.diff
	Fri, Mar 21, 4:10 PM

	F15409709: D10775.id25865.diff
	Wed, Mar 19, 4:54 AM

	F15382812: D10775.id25860.diff
	Fri, Mar 14, 2:34 PM

	F15377416: D10775.id25865.diff
	Thu, Mar 13, 8:48 AM

	F15360427: D10775.id25860.diff
	Tue, Mar 11, 8:35 AM

	F15346237: D10775.id.diff
	Mon, Mar 10, 1:51 PM

	F15342859: D10775.id25860.diff
	Sun, Mar 9, 9:33 PM

Subscribers

Details

Reviewers

Maniphest Tasks

T3942: Make messaging for 100+ click-to-load changeset behavior more clear

Commits

Restricted Diffusion Commit
rPf5235f7e09eb: Make loading large changesets slightly easier

Summary

Fixes T3942, turns the load links into buttons.

Test Plan

Set my limit to 1, test page.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 25860.Nov 3 2014, 12:01 AM

chad retitled this revision from to Make loading large changesets slightly easier.

chad updated this object.

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

chad added a reviewer: epriestley.

chad added a task: T3942: Make messaging for 100+ click-to-load changeset behavior more clear.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptNov 3 2014, 12:01 AM

Screen_Shot_2014-11-02_at_3.56.51_PM.png (440×1 px, 53 KB)

Harbormaster completed remote builds in B2933: Diff 25860.Nov 3 2014, 12:02 AM

epriestley accepted this revision.Nov 3 2014, 12:58 PM

epriestley edited edge metadata.

epriestley added inline comments.

src/applications/differential/controller/DifferentialRevisionViewController.php
143	The "Diff" language is technically accurate: a revision may have several diffs, some of which are small and some of which are large. Only the large ones would trigger this prompt.

This revision is now accepted and ready to land.Nov 3 2014, 12:58 PM

fixed language

Closed by commit rPf5235f7e09eb: Make loading large changesets slightly easier (authored by chad, committed by chad). · Explain WhyNov 3 2014, 5:31 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

controller/

DifferentialRevisionViewController.php

5 lines

view/

DifferentialChangesetListView.php

3 lines

Diff 25865

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...

src/applications/differential/view/DifferentialChangesetListView.php

Loading...