Closed Bug 1157560 Opened 9 years ago Closed 9 years ago

[APZ] Popup arrow panel UI broken if scrollbar in rtl displayed

Categories

(Core :: Panning and Zooming, defect)

40 Branch
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1194851
Tracking Status
firefox40 --- affected

People

(Reporter: alice0775, Unassigned)

References

Details

(Keywords: regression, Whiteboard: [gfx-noted])

Attachments

(1 file)

Attached image screenshot
This was caused by https://hg.mozilla.org/mozilla-central/rev/17aad8f83237
And fixed by backed out https://hg.mozilla.org/mozilla-central/rev/570bb53a3e7b

Steps to Reproduce:
1. Install https://github.com/gorhill/uBlock/releases/download/0.9.4.6-dev.1/uBlock0.firefox.xpi
2. Turn on "I am an advanced user (Required reading)" from option ( chrome://ublock0/content/dashboard.html )
3. Open Youtube Video (e.g. https://www.youtube.com/watch?v=55s3T7VRQSc )
4. Click uBlock0 toolbar button

Actual Results:
1 columun is missing

Expected Results:
3 columns should display
Regression window if forced to enable APZ (layers.async-pan-zoom.enabled = true):
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=24fe1880f83b&tochange=d3b41b0ead7d

Triggered by: d3b41b0ead7d David Anderson — Remove apz.subframe.enabled and assume it is always true. (bug 1128761, r=kats)
Something to check: does this still happen if layers.async-pan-zoom.enabled is false but layout.event-regions.enabled is true?
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #2)
> Something to check: does this still happen if layers.async-pan-zoom.enabled
> is false but layout.event-regions.enabled is true?

setting
layers.async-pan-zoom.enabled = false
layout.event-regions.enabled = true (default)
in Nightly40.0a1(2015-04-22 946ac85af8f4)
and restart, then the problem is gone.
Problem no longer reproducible with 20150426 nightly.
Fixed ?
yes, it was fixed by backed out (https://hg.mozilla.org/mozilla-central/rev/570bb53a3e7b)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
Reopening because we'll need to fix this at some point, apz will be on by default soon.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
I can reproduce this on Linux using the latest uBlock add-on from github and the rest of the STR in comment 0. Looks like the clip isn't properly accounting for the scrollbar on the left, although I haven't looked at a layer dump to verify that yet.
Whiteboard: [gfx-noted]
Forward-duping since the newer bug has a simple test case attached.
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: