mozilla-mobile / mozilla-vpn-client

A fast, secure and easy to use VPN. Built by the makers of Firefox.
https://vpn.mozilla.org
Other
462 stars 113 forks source link

Page don’t scrolled up completely at countries expand #4535

Closed data-sync-user closed 1 year ago

data-sync-user commented 2 years ago

Mozilla VPN affected versions:

Platform tested:

Prerequisites:

Steps to reproduce:

  1. Open the VPN app;
  2. Click to select a location; (i.e Poland - Warsaw)
  3. Scroll down the list until the “USA” location;
  4. Click to expand the list of cities from that location;
  5. Observe the behavior;

Expected results:

Actual results:

!image-20220927-063919.png|width=466,height=842!

┆Issue is synchronized with this Jira Bug ┆Reporter: Valentina Virlics

data-sync-user commented 2 years ago

➤ Andrea Marchesini commented:

Can we close this as work-as-expected? CC Peter Benvenuto

data-sync-user commented 2 years ago

➤ Valentina Virlics commented:

Andrea Marchesini Peter Benvenuto This is also happening if the list of the cities is one item long.

Not something major or important, but would expect that while doing the expand action, the cities to be revealed not covered by the nav bar.

Attaching a video for clearance

!noscroll.gif|width=476,height=842!

.

data-sync-user commented 2 years ago

➤ Peter Benvenuto commented:

Andrea Marchesini Valentina Virlics I was actually speaking with matt about this, the page should scroll up so that there the content is not hidden. Either that or a consistent amount of padding added to the bottom of each screen so that nothing is ever hidden by the nav bar.

data-sync-user commented 1 year ago

➤ Lesley Norton commented:

Cancelling - For some countries like the US, the height of the list of server cities is greater than the available height of the app. When this happens, the name of the country scrolls to the top so that we show as much of the city list as possible and the rest is accessible by scrolling.

data-sync-user commented 5 months ago

➤ Santiago Andrigo commented:

Moving back to Cancelled