Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

Web content that is sticky to the bottom is hidden until refresh #5884

Closed
jonalmeida opened this issue Nov 16, 2021 · 2 comments
Closed

Web content that is sticky to the bottom is hidden until refresh #5884

jonalmeida opened this issue Nov 16, 2021 · 2 comments
Assignees
@ionutbedregeanu
Labels
bug dynamic-toolbar Issues related to using a dynamic toolbar QAVerified
Projects
Android Focus

Comments

@jonalmeida
Copy link
Contributor

Steps to reproduce

  1. Navigate to reddit.com
  2. Scroll the toolbar away before the page loaded.
  3. Scroll the toolbar away after the page is loaded.

(see examples below)

Expected behavior

  • At step 2, the toolbar is made visible and persists in that position with the full viewport visible after the page is done loading.
  • At step 3, the the toolbar is made visible and persists in that position with the full viewport visible after a touch event is recorded.

Expected results are seen in Fenix:

device-2021-11-16-104751.mp4
device-2021-11-16-105103.mp4

Actual behavior

  • At step 2 and step 3, the toolbar is still hidden, the viewport is visible, but scrolling then locks the toolbar visible and cuts out the bottom viewport.
device-2021-11-16-105321.mp4

Device information

  • Android device: All devices
  • Focus version: Firefox Nightly 96
@jonalmeida jonalmeida added bug dynamic-toolbar Issues related to using a dynamic toolbar labels Nov 16, 2021
@jonalmeida jonalmeida added this to Backlog in Android Focus via automation Nov 16, 2021
@ionutbedregeanu ionutbedregeanu self-assigned this Nov 22, 2021
@ionutbedregeanu ionutbedregeanu moved this from Backlog to In Progress in Android Focus Nov 22, 2021
ionutbedregeanu pushed a commit to ionutbedregeanu/focus-android that referenced this issue Nov 23, 2021
For mozilla-mobile#5884 - Use the correct value of toolbar height at …
7ff6777 
…dynamic behavior
ionutbedregeanu pushed a commit to ionutbedregeanu/focus-android that referenced this issue Nov 23, 2021
For mozilla-mobile#5884 - Use the correct value of toolbar height at …
f7152b8 
…dynamic behavior
@ionutbedregeanu ionutbedregeanu mentioned this issue Nov 23, 2021
Merged
@Mugurell Mugurell mentioned this issue Nov 23, 2021
Closed
mergify bot pushed a commit that referenced this issue Nov 23, 2021
@mergify
For #5884 - Use the correct value of toolbar height at dynamic behavior
e4276ca
@ionutbedregeanu ionutbedregeanu added the QAReady flag for test read & needed label Nov 23, 2021
@ionutbedregeanu ionutbedregeanu mentioned this issue Nov 24, 2021
Closed
@lobontiumira
Copy link

Verified as fixed on the latest Focus Nightly from 11/24 (build 353281028 GV 96.0a1-20211123094249) with Google Pixel (Android 10):

5884.mp4

Android Focus automation moved this from In Progress to Done Nov 24, 2021
@lobontiumira lobontiumira added QAVerified and removed QAReady flag for test read & needed labels Nov 24, 2021
iorgamgabriel pushed a commit to iorgamgabriel/focus-android that referenced this issue Dec 3, 2021
@iorgamgabriel
For mozilla-mobile#5884 - Use the correct value of toolbar height at …
53d62c6 
…dynamic behavior
iorgamgabriel pushed a commit to iorgamgabriel/focus-android that referenced this issue Dec 3, 2021
@iorgamgabriel
For mozilla-mobile#5884 - Use the correct value of toolbar height at …
33fc031 
…dynamic behavior
mergify bot pushed a commit that referenced this issue Dec 15, 2021
For #5884 - Use the correct value of toolbar height at dynamic behavior
254529d 
(cherry picked from commit e4276ca)
Amejia481 pushed a commit that referenced this issue Dec 15, 2021
@mergify
For #5884 - Use the correct value of toolbar height at dynamic behavi…
eb01036 
…or (#6077)

(cherry picked from commit e4276ca)

Co-authored-by: Ionut Cristian Bedregeanu <ionut.bedregeanu@softvision.ro>
@lobontiumira
Copy link

Verified as fixed on the RC 95.2.0 with Lenovo tablet M10 (Android 10).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ionutbedregeanu ionutbedregeanu

Labels
bug dynamic-toolbar Issues related to using a dynamic toolbar QAVerified
Projects
Android Focus
  
Done
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jonalmeida @ionutbedregeanu @lobontiumira