Closed Bug 1649865 Opened 5 years ago Closed 5 years ago

Can't scroll to the bottom of a bugzilla bug in Fenix

Categories

(GeckoView :: General, defect, P1)

Unspecified
All
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jesup, Assigned: snorp)

References

Details

(Whiteboard: [geckoview:m81][geckoview:m82][geckoview:m83][geckoview:toolbar][geckoview:m84])

Attachments

(1 file)

I was trying to comment on bug 1599160 in Fenix Nightly, and I couldn't get to the bottom buttons no matter what I did (desktop mode, rotate, reload, kill and restart, etc). I could get to them in other (shorter?) bugs.

ni? kbrosnan to try with reference browser

Flags: needinfo?(kbrosnan)

Bug 1650644 is similar (possibly a dupe, not sure yet) and there is some investigation ongoing there.

See Also: → 1650644

I wonder whether this is a recent regression or not.

Randell, are you able to reproduce the issue on bugzilla pages where the content's width totally fits the screen width?

I was wondering why I can't see the issue on my environments (pixel 3 and an Android emulator), I noticed that in the screenshot in comment 0 there is a horizontal overflow area, so I guess the screenshot was taken after zooming in to some extent? (And I presume you couldn't zoom out to the original scale size due to bug 1652190)

Flags: needinfo?(rjesup)

Loading this bug I can see the bottom buttons. If I select the Add Comment area, I can no longer see them even if I delete everything, dismiss the keyboard, etc. Killing the tab and reloading the URL restores the buttons.

Flags: needinfo?(rjesup)

Thank you! That explains this is a dup of bug 1652190. When you click to the comment area, zoomToRect happens, which means the content is zoomed in to some extent.

Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(kbrosnan)
Resolution: --- → DUPLICATE
Status: RESOLVED → REOPENED
Resolution: DUPLICATE → ---

(In reply to Kevin Brosnan [:kbrosnan] from comment #8)

This does not seem to be fixed by bug 1652190. https://github.com/mozilla-mobile/fenix/issues/13055 and https://github.com/mozilla-mobile/fenix/issues/13198

I think there are multiple things going on in those Fenix issues:

  • Not being able to interact with a button that was previously underneath the toolbar. I expect this was fixed recently by bug 1652288.
  • Not being able to hide the toolbar in the first place, to reveal content underneath. The cause of this probably varies from page to page.
    • Bug 1650644 should fix some cases of this bug (in particular, cases where the page is not otherwise scrollable).
    • Bug 1633322 should fix other cases of this bug (in particular, cases that involve subframe scrolling or non-passive touch event listeners).
    • I am hopeful that between them, bug 1650644 and bug 1633322 fix all cases of this, but it's possible there are other issues we are overlooking.
Depends on: 1652288, 1650644, 1633322
See Also: 1650644
Assignee: nobody → snorp
Severity: -- → S2
Priority: -- → P1
Whiteboard: [geckoview:m81]
Whiteboard: [geckoview:m81] → [geckoview:m81][geckoview:m82]
Whiteboard: [geckoview:m81][geckoview:m82] → [geckoview:m81][geckoview:m82][geckoview:m83]
Whiteboard: [geckoview:m81][geckoview:m82][geckoview:m83] → [geckoview:m81][geckoview:m82][geckoview:m83][geckoview:toolbar]
Whiteboard: [geckoview:m81][geckoview:m82][geckoview:m83][geckoview:toolbar] → [geckoview:m81][geckoview:m82][geckoview:m83][geckoview:toolbar][geckoview:m84]

I wonder whether bug 1663000 fixed this or not.

Randell, can you please check whether this issue still persists on Fenix nightly? I've confirmed bug 1663000 is included in the latest nightly.

Flags: needinfo?(rjesup)

This does appear to be fixed. Thanks

Flags: needinfo?(rjesup)

Thank you!

Status: REOPENED → RESOLVED
Closed: 5 years ago5 years ago
Resolution: --- → FIXED

Moving toolbar bugs to the new GeckoView::Toolbar component.

Component: General → Toolbar
Component: Toolbar → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: