Summary

Add controls to select base and tip commits in the RB.DiffCommitListView

Review Request #10132 — Created Aug. 30, 2018 and submitted Oct. 18, 2018, 9:39 p.m.

Information

Owner

brennie

Repository

Review Board

Branch

release-4.0.x

Bugs

Depends On

~~10128~~

Blocks

10136

Commit

3391784...

Reviewers

Groups

reviewboard

People

Description

The RB.DiffCommitListView can now optionally display controls for
selecting between base and tip commits to control what will be displayed
in the diffviewer. The actual logic for changing the diffs is not yet
implemented -- just the control itself.

Testing Done

Ran JS tests.

Files

Issues

Description	From	Last Updated
Screenshot?	david	Sept. 27, 2018, 9:39 a.m.
Col: 36 Missing semicolon.	reviewbot	Aug. 30, 2018, 6:10 p.m.
Col: 55 Missing semicolon.	reviewbot	Sept. 2, 2018, 11:27 a.m.
Col: 36 Missing semicolon.	reviewbot	Sept. 2, 2018, 11:27 a.m.
Col: 55 Missing semicolon.	reviewbot	Sept. 2, 2018, 11:27 a.m.
Col: 36 Missing semicolon.	reviewbot	Sept. 2, 2018, 11:27 a.m.
Col: 55 Missing semicolon.	reviewbot	Sept. 2, 2018, 11:27 a.m.
Col: 36 Missing semicolon.	reviewbot	Sept. 2, 2018, 11:27 a.m.
Alphabetical order.	chipx86	Sept. 4, 2018, 1:06 p.m.
If there is no parent in the collection.	chipx86	Sept. 4, 2018, 1:06 p.m.
Indentation is incorrect for the comment.	chipx86	Sept. 4, 2018, 1:06 p.m.
Alphabetical order.	chipx86	Sept. 4, 2018, 1:06 p.m.
Alphabetical order.	chipx86	Sept. 4, 2018, 1:52 p.m.
Alphabetical order.	chipx86	Sept. 4, 2018, 1:52 p.m.
"Option Args"	chipx86	Sept. 4, 2018, 1:07 p.m.
Indentation is wrong. This would fit on one line though.	chipx86	Sept. 4, 2018, 1:07 p.m.
Parents around the ternary. The ? and : should also align with tipCommitID.	chipx86	Sept. 4, 2018, 1:07 p.m.
You can start the first parameter on the listenTo line.	chipx86	Sept. 4, 2018, 1:08 p.m.
Missing docs.	chipx86	Sept. 4, 2018, 1:09 p.m.
This looks like leftover debugging? Maybe just put a TODO here, or leave out the function until it's implemented.	chipx86	Sept. 4, 2018, 1:09 p.m.

flake8 passed.

JSHint failed.

JSHint

reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js (Diff revision 1)
The issue has been resolved. Show all issues
```
Col: 36
 Missing semicolon.
```

Change Summary:

jshint

Commit:

d94e73216a2731f8145db559c678ed794548eed2

dd6049fdc52410a490dbaec773c9fc57ec42bee2

Diff:

Revision 2 (+466 -46)

Show changes

	reviewboard/diffviewer/models/diffcommit.py
	reviewboard/static/rb/css/pages/review-request.less
	reviewboard/static/rb/js/diffviewer/models/diffCommit.es6.js
	reviewboard/static/rb/js/diffviewer/models/diffCommitListModel.es6.js
	reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js
	reviewboard/static/rb/js/diffviewer/views/tests/diffCommitListViewTests.es6.js
	reviewboard/static/rb/js/pages/views/diffViewerPageView.es6.js

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Change Summary:

Rebase.

Commit:

dd6049fdc52410a490dbaec773c9fc57ec42bee2

40d70a155fabe498aa9da5727b3e0d500b6584a2

Diff:

Revision 3 (+465 -42)

Show changes

	reviewboard/diffviewer/models/diffcommit.py
	reviewboard/static/rb/css/pages/review-request.less
	reviewboard/static/rb/js/diffviewer/models/diffCommit.es6.js
	reviewboard/static/rb/js/diffviewer/models/diffCommitListModel.es6.js
	reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js
	reviewboard/static/rb/js/diffviewer/views/tests/diffCommitListViewTests.es6.js
	reviewboard/static/rb/js/pages/views/diffViewerPageView.es6.js

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Change Summary:

Use PKs over commit IDs

Commit:

40d70a155fabe498aa9da5727b3e0d500b6584a2

b8dbaac0cdab0f09f31d5cd4664940a8852cdfce

Diff:

Revision 4 (+501 -42)

Show changes

	reviewboard/diffviewer/models/diffcommit.py
	reviewboard/static/rb/css/pages/review-request.less
	reviewboard/static/rb/js/diffviewer/collections/diffCommitCollection.es6.js
	reviewboard/static/rb/js/diffviewer/models/diffCommit.es6.js
	reviewboard/static/rb/js/diffviewer/models/diffCommitListModel.es6.js
	reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js
	reviewboard/static/rb/js/diffviewer/views/tests/diffCommitListViewTests.es6.js
	reviewboard/static/rb/js/pages/views/diffViewerPageView.es6.js

Checks run (1 failed, 1 succeeded)

flake8 passed.

JSHint failed.

JSHint

reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js (Diff revision 4)
The issue has been resolved. Show all issues
```
Col: 55
 Missing semicolon.
```
reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js (Diff revision 4)
The issue has been resolved. Show all issues
```
Col: 36
 Missing semicolon.
```

Change Summary:

CommitId -> CommitID

Commit:

b8dbaac0cdab0f09f31d5cd4664940a8852cdfce

a8d48b089479dad6e428c256a5407aae78668636

Diff:

Revision 5 (+501 -42)

Show changes

	reviewboard/diffviewer/models/diffcommit.py
	reviewboard/static/rb/css/pages/review-request.less
	reviewboard/static/rb/js/diffviewer/collections/diffCommitCollection.es6.js
	reviewboard/static/rb/js/diffviewer/models/diffCommit.es6.js
	reviewboard/static/rb/js/diffviewer/models/diffCommitListModel.es6.js
	reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js
	reviewboard/static/rb/js/diffviewer/views/tests/diffCommitListViewTests.es6.js
	reviewboard/static/rb/js/pages/views/diffViewerPageView.es6.js

Checks run (1 failed, 1 succeeded)

flake8 passed.

JSHint failed.

JSHint

reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 55
 Missing semicolon.
```
reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js (Diff revision 5)
The issue has been resolved. Show all issues
```
Col: 36
 Missing semicolon.
```

Change Summary:

CommitId -> CommitID

Commit:

a8d48b089479dad6e428c256a5407aae78668636

5d7974aaa61deb8c8a3e0f8b069b1f6c10c4df07

Diff:

Revision 6 (+502 -42)

Show changes

	reviewboard/diffviewer/models/diffcommit.py
	reviewboard/static/rb/css/pages/review-request.less
	reviewboard/static/rb/js/diffviewer/collections/diffCommitCollection.es6.js
	reviewboard/static/rb/js/diffviewer/models/diffCommit.es6.js
	reviewboard/static/rb/js/diffviewer/models/diffCommitListModel.es6.js
	reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js
	reviewboard/static/rb/js/diffviewer/views/tests/diffCommitListViewTests.es6.js
	reviewboard/static/rb/js/pages/views/diffViewerPageView.es6.js

Checks run (1 failed, 1 succeeded)

flake8 passed.

JSHint failed.

JSHint

reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 55
 Missing semicolon.
```
reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js (Diff revision 6)
The issue has been resolved. Show all issues
```
Col: 36
 Missing semicolon.
```

Change Summary:

jshint

Commit:

5d7974aaa61deb8c8a3e0f8b069b1f6c10c4df07

7dc3256855ee1bad9194f4f2a097d42dc239ecf0

Diff:

Revision 7 (+502 -42)

Show changes

	reviewboard/diffviewer/models/diffcommit.py
	reviewboard/static/rb/css/pages/review-request.less
	reviewboard/static/rb/js/diffviewer/collections/diffCommitCollection.es6.js
	reviewboard/static/rb/js/diffviewer/models/diffCommit.es6.js
	reviewboard/static/rb/js/diffviewer/models/diffCommitListModel.es6.js
	reviewboard/static/rb/js/diffviewer/views/diffCommitListView.es6.js
	reviewboard/static/rb/js/diffviewer/views/tests/diffCommitListViewTests.es6.js
	reviewboard/static/rb/js/pages/views/diffViewerPageView.es6.js

Checks run (2 succeeded)

flake8 passed.

JSHint passed.