Add the ability to use next and previous buttons for SlideshowView.
Review Request #13165 — Created July 31, 2023 and submitted
Information | |
---|---|
david | |
Review Board | |
release-6.x | |
Reviewers | |
reviewboard | |
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.
-
-
reviewboard/static/rb/css/ui/slideshow.less (Diff revision 1) 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).
-
Commits: |
|
|||||||||
---|---|---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+130 -20) |
Checks run (2 succeeded)
flake8
passed.
JSHint
passed.