1859671 - Fakespot bottom sheet's section should remember their expanded state for the current page

Status: VERIFIED FIXED

(Fenix :: Shopping, enhancement, P2)

Description

[Chris Peterson [:cpeterson]]

Steps to reproduce

1. Load a PDP.
2. Open the Fakespot bottom sheet.
3. Expand the "Show more" highlights, "How we determine review quality", and "Settings" sections.
4. Close the bottom sheet.
5. Reopen the bottom sheet.

Expected behavior

The "Show more" highlights, "How we determine review quality", and "Settings" sections should remember whether they were expanded or closed on the current page.

Actual behavior

The "Show more" highlights, "How we determine review quality", and "Settings" sections are all closed.

Comment 1

[Rahul Sainani [:rsainani]]

I think the current behavior where the user opens the bottom sheet and the cards are in collapsed state is correct. We shouldn't persist the expanded states of each card, it would lead to not being able to see the product recommendation and would be a higher overhead to persist this information.

Comment 2

[Chris Peterson [:cpeterson]]

Verdi says we should remember the bottom sheet sections' expanded state as long the user is on the page. If you load another page in that tab then we can forget. We don’t need to remember across app restarts or if the user opens a second tab for the same product page.

Comment 3

[BMO Github Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1859671 - Remember review checker card states for the session (#4457)

Comment 4

[BMO Github Automation]

Authored by https://github.com/rahulsainani
https://github.com/mozilla-mobile/firefox-android/commit/c4ede23266b730e7c84c3ef1edfeab932a2ec9b2
[main] Bug 1859671 - Remember review checker card states for the session

Comment 5

[Adina Petridean]

Attachment: expand.mp4

Comment 6

[Adina Petridean]

Verified as fixed on Nightly 121.0a1 from 11/16 with Motorola G9 Plus (Android 11) and Sony Xperia (Android 6.0.1).