Page MenuHomePhabricator

Snackbars should appear above BottomNavigationView.
Closed, ResolvedPublic

Description

When one or more reading lists are deleted (for example), a snackbar appears for a rather long time that obscures the bottom navigation bar and prevents the user from navigating to a different fragment until the snackbar disappears. Snackbars should ideally appear above the bottom navigation component.

Event Timeline

Note: having more than one action in a Snackbar is against Material guidelines.
https://material.io/guidelines/components/snackbars-toasts.html#snackbars-toasts-usage

A more proper implementation might be to have the snackbar appear on top of the navigation bar.

Dbrant renamed this task from Add a dismiss button to the Reading list delete notification dialog to Snackbars should appear above BottomNavigationView..Mar 6 2017, 3:38 PM
Dbrant claimed this task.
Dbrant updated the task description. (Show Details)

Change 341338 had a related patch set uploaded (by dbrant):
[apps/android/wikipedia] Make SnackBars appear above BottomNavigationView.

https://gerrit.wikimedia.org/r/341338

Change 341338 merged by jenkins-bot:
[apps/android/wikipedia] Make SnackBars appear above BottomNavigationView.

https://gerrit.wikimedia.org/r/341338

For QA:

  • Go to the Feed, and cause a Snackbar to appear (e.g. add one of the articles to a reading list).
  • Make sure that the Snackbar appears above the bottom navigation buttons.
ABorbaWMF subscribed.

Appears to be fixed. Tested on Nexus 4 with Android 5.1 and a Pixel with Android 7.1.1 and the Alpha App 2.5.190-alpha-2017-03-08