Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

www.engadget.com / www.techcrunch.com - header bar floating away(not fixed position) #1560

Closed
bull500 opened this issue Aug 19, 2015 · 6 comments
Labels
browser-fixme This requires manual assignment for the browser name
Milestone

Comments

@bull500
Copy link

bull500 commented Aug 19, 2015

URL: http://www.engadget.com/2015/08/19/cochin-airport-runs-on-solar-power/
Browser / Version: Firefox Mobile 41.0
Operating System: Android 5.0.2
Problem type: Layout is messed up

Steps to Reproduce

  1. Navigate to: http://www.engadget.com/2015/08/19/cochin-airport-runs-on-solar-power/
  2. start scrolling down slowly
  3. the fixed grey header bar shifts it's position for a certain distance and then returns to the top area after a certain scroll distance.

Expected Behavior:
Header bar should stay fixed floating on the top

Actual Behavior:
Header bar shifts it's position for some scroll distance.

@bull500 bull500 changed the title www.engadget.com - layout is messed up www.engadget.com / www.techcrunch.com - header bar floating away(not fixed position) Sep 7, 2015
@bull500
Copy link
Author

bull500 commented Sep 7, 2015

screenshot_2015-09-07-15-06-29
screenshot_2015-09-07-15-09-06

For the techcrunch problem, you'll have to fling scroll to notice the issue.
Engadget needs the user to scroll slowly for the top few top contents of the page.

@hallvors
Copy link

The device may be throttling some scroll events in order to save some CPU time and make the page itself scroll more smoothly. I've seen similar effects on Facebook. I think it's an acceptable trade-off to have the rest of the page painted better and the scrolling be smoother - the top navigation will slot into place when those scroll events fire and the page updates the styling.

(This is a bit of guesswork though - if the element is set to position:fixed we may want a smarter scroll algorithm)

@bull500
Copy link
Author

bull500 commented Oct 12, 2015

@hallvors just found a temp fix solution from IRC - disabling Fullscreen(Settings > Display > Full screen browsing) browsing fixes this.
Something about dynamic toolbar causing this trouble.

More and more sites seem to be using this method of top header bar menu.

Wish fullscreen has this bug fixed soon.

@bull500
Copy link
Author

bull500 commented Nov 17, 2015

@hallvors this issue has been fixed in Nightly 45 on android as per bug 1214267

@hallvors
Copy link

hallvors commented Jan 5, 2016

Thanks @bull500 - so it was indeed a position:fixed core problem ;)

@bull500
Copy link
Author

bull500 commented Jan 5, 2016

@hallvors happy to help. :)
Glad this got fixed before the site redesign. It was an isolated/special case.

@karlcow karlcow added this to the fixed milestone Oct 30, 2017
@karlcow karlcow added the browser-fixme This requires manual assignment for the browser name label Jun 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser-fixme This requires manual assignment for the browser name
Projects
None yet
Development

No branches or pull requests

4 participants