Summary:

Add the ability to use next and previous buttons for SlideshowView.

Review Request #13165 — Created July 31, 2023 and submitted Aug. 3, 2023, 12:18 p.m.

Information
Owner:	david
Repository:	Review Board
Branch:	release-6.x
Bugs:
Depends On:
Reviewers
Groups:	reviewboard
People:

Description

The SlideshowView implementation ported over from rb-website assumed
that the nav was basically a table of contents, with a nav item for each
slide. This change adds functionality to use next/previous buttons
instead (you cannot currently use both).

Testing Done

Used in conjunction with a change that adds a slideshow to the review
dialog.

Commits

Summary
	Add the ability to use next and previous buttons for SlideshowView.

Screenshots

Issues

Description	From	Last Updated
Can you have the "Alternatively, ..." align within the structure, in its own paragraph? This should all be under the …	chipx86	Aug. 2, 2023, 9:50 a.m.
Missing types.	chipx86	Aug. 2, 2023, 9:51 a.m.

flake8 passed.

JSHint passed.

reviewboard/static/rb/css/ui/slideshow.less (Diff revision 1)

Show all issues

Can you have the "Alternatively, ..." align within the structure, in its own paragraph? This should all be under the "Structure" header to format right (... ok it currently doesn't render anywhere, but I have some prototype code sitting around for this).

reviewboard/static/rb/js/ui/views/slideshowView.ts (Diff revision 1)
Show all issues
```
Missing types.
```

Commits:

	Summary
-		Add the ability to use next and previous buttons for SlideshowView.
+		Add the ability to use next and previous buttons for SlideshowView.

Diff:

Revision 2 (+130 -20)

Show changes

	reviewboard/static/rb/css/ui/slideshow.less
	reviewboard/static/rb/js/ui/views/slideshowView.ts

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-6.x (c2cf6c6)