1541158 - Make changes to full zoom on RDM pages pass through to RDM document

Status: VERIFIED FIXED

(DevTools :: Responsive Design Mode, defect, P1)

Description

[Brad Werth [:bradwerth]]

Steps to Reproduce:

1. Open any page.
2. Toggle on Responsive Design Mode.
3. Change the full zoom level.

Actual Results: the UI changes zoom levels
Expected Results: the RDM document should change zoom levels

Comment 1

[Brad Werth [:bradwerth]]

Attachment: Bug 1541158 Part 1: Passthrough zoom to RDM document.

This ensures that any changes to zoom level, when RDM is active,
happen on the content document and not on the RDM interface.

Comment 2

[Brad Werth [:bradwerth]]

Attachment: Bug 1541158 Part 2: Add a test that only RDM content is affected by full zoom changes.

Depends on D26040

Comment 3

[Brad Werth [:bradwerth]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=7c9e0c7b86ffe7f23907792efeb2384026cb9486

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fce9f2905ff0
Part 1: Passthrough zoom to RDM document. r=pbro
https://hg.mozilla.org/integration/autoland/rev/efc734d0f506
Part 2: Add a test that only RDM content is affected by full zoom changes. r=pbro

Comment 6

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/fce9f2905ff0
https://hg.mozilla.org/mozilla-central/rev/efc734d0f506

Comment 8

[Rares Doghi]

Hi Guys, When the user Zooms in using the CTRL + Scroll Wheel from the mouse the entire Page zooms in and out including the RDM UI bar but if we Use CTRL + +/- then only the content zooms in and out the UI bar remains at 100%.
Is this the correct behavior ? Is it intended for the entire page to zoom in and out when using the Mouse wheel ?

Comment 9

[Patrick Brosset <:pbro>]

No this behavior is not correct. This is still an open bug that is currently tracked and worked on in bug 1516413.

Comment 10

[Rares Doghi]

This issue is Verified as Fixed in 69.0a1 (2019-06-03).