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

Review Request #13165 — Created July 31, 2023 and submitted — Latest diff uploaded

Information

Review Board
release-6.x

Reviewers

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).

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

Diff Revision 1

This is not the most recent revision of the diff. The latest diff is revision 2. See what's changed.

orig
1
2

Commits

First Last Summary ID Author
Add the ability to use next and previous buttons for SlideshowView.
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.
f1e9f279bc23da5f9caf1e1b30dc2143ef00952b David Trowbridge
reviewboard/static/rb/css/ui/slideshow.less
reviewboard/static/rb/js/ui/views/slideshowView.ts
Loading...