Differential D21410

Introduce "DiffusionCommitGraphView", which unifies "HistoryListView" and "HistoryTableView"
ClosedPublic
Actions

Authored by epriestley on Jul 10 2020, 6:20 PM.

Tags

None

Referenced Files

	F13182120: D21410.diff
	Thu, May 9, 7:39 PM

	Unknown Object (File)
	Tue, May 7, 4:13 AM

	Unknown Object (File)
	Fri, May 3, 8:21 AM

	Unknown Object (File)
	Sat, Apr 27, 7:46 PM

	Unknown Object (File)
	Thu, Apr 25, 2:18 AM

	Unknown Object (File)
	Tue, Apr 16, 7:23 PM

	Unknown Object (File)
	Thu, Apr 11, 9:41 AM

	Unknown Object (File)
	Apr 2 2024, 10:58 PM

View All 43 Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13552: "Close Revision" actions are executed by the Message worker, and do not execute if a previously discovered commit becomes reachable from a permanent branch
Commits: rP46695c76ebfe: Introduce "DiffusionCommitGraphView", which unifies "HistoryListView" and…

Summary

Ref T13552. Currently, commit lists are sometimes rendered as an object list and sometimes rendered as a table. There are two separate views for table rendering.

Add a fourth view ("list, with a graph") with the eventual intent of unifying all the other views. For now, this only replaces "HistoryListView" -- and needs some more work to really be a convincing replacement.

Test Plan

Looked at "History" in Diffusion, saw an ugly view with all the information we want.
Grepped for "HistoryListView", no hits.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jul 10 2020, 6:20 PM

Harbormaster completed remote builds in B24772: Diff 50953.Jul 10 2020, 6:22 PM

epriestley requested review of this revision.Jul 10 2020, 6:22 PM

This revision was not accepted when it landed; it landed in state Needs Review.Aug 12 2020, 3:59 PM

Closed by commit rP46695c76ebfe: Introduce "DiffusionCommitGraphView", which unifies "HistoryListView" and… (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP46695c76ebfe: Introduce "DiffusionCommitGraphView", which unifies "HistoryListView" and….

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

applications/

diffusion/

controller/

DiffusionHistoryController.php

25 lines

view/

DiffusionCommitGraphView.php

415 lines

DiffusionHistoryListView.php

Diff 51027

src/__phutil_library_map__.php

Loading...

src/applications/diffusion/controller/DiffusionHistoryController.php

Loading...

src/applications/diffusion/view/DiffusionCommitGraphView.php

Loading...

src/applications/diffusion/view/DiffusionHistoryListView.php

Loading...