1126526 - Don't show the navigation arrows for the screenshots if all of them are visible on the details page

Status: VERIFIED FIXED

(Marketplace Graveyard :: Consumer Pages, defect, P3)

Description

[krupa raj[:krupa]]

Attachment: navigation arrows.png

steps to reproduce:
1. Load https://marketplace-dev.allizom.org/app/book-note?src=branded-editorial-element
2. Notice the preview screenshot section

expected behavior:
Since there are only two screenshots listed, there are no navigation arrows


actual behavior:
Navigation arrows are present for just two screenshots and are redundant.

Comment 1

[David Durst [:ddurst]]

This bug would appear to actually be that, when there are fewer than 3 screenshots, the preview tray should show the first one as current and the second one to the right, rather than the second one as current and the first one to the left.

Comment 2

[Davor Spasovski [:spasovski]]

https://github.com/mozilla/fireplace/commit/88bf48f

The desktop detail screenshots previews will now start with the first one "centered" when less than 3 previews exist. A good app to test this with is: https://marketplace-dev.allizom.org/app/slice-fruits

Comment 3

[krupa raj[:krupa]]

verified fixed at https://marketplace-dev.allizom.org/app/book-note?src=branded-editorial-element