Fenix: FNX-5584 ⁃ [Bug] Opening images in new tab results in tiny images.

[Lululalu]

Steps to reproduce

Go to a webpage that contains images (in my case it happens with PNG images)
Long press the image
Open the a new tab

Expected behavior

The imagine is opened in a new tab in normal size.

Actual behavior

The image is pixels tiny. Only after touching the address bar and back, the imagine is shown properly.

Device information

• Android device: Galaxy A5 2017 8.0
• Fenix version: 1.0.1919

[vesta0]

@cpeterso wondering if this is GV related?

[cpeterso]

I can reproduce this bug in both Fenix and R-B, so I filed a GV bug:

https://bugzilla.mozilla.org/show_bug.cgi?id=1551659

[vesta0]

Thank you @cpeterso !

[sblatz]

Is this still a p1? If so, let's make sure it's properly prioritized in GV.

cc: @vesta0, @cpeterso

[cpeterso]

Is this still a p1? If so, let's make sure it's properly prioritized in GV.

This bug is on the GV team's July backlog, but we haven't started investigating yet. This bug doesn't seem like a P1 to me. :)

[vesta0]

This is a P1. We wanted this fixed before MVP but that was not possible so we removed the "view image" option from the image toolbar as a workaround . @cpeterso @bifleming please make sure this is not de-prioritized now that we have a different labeling system.

[bifleming]

Hi Vesta, it's currently #2 in the priority list.

On Mon, Jul 22, 2019 at 9:48 PM Vesta Zare [e-mail] wrote:

This is a P1. We wanted this fixed before MVP but that was not possible so
we removed the "view image" option from the image toolbar as a workaround .
@cpeterso https://github.com/cpeterso @bifleming
https://github.com/bifleming please make sure this is not
de-prioritized now that we have a different labeling system.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mozilla-mobile/fenix/issues/2312?email_source=notifications&email_token=ALLADYFT6ZFS6ORU4PRJ4RDQA2ERJA5CNFSM4HLFOHJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2R5C5A#issuecomment-514052468,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALLADYAIHSWN52FKCD4IZKDQA2ERJANCNFSM4HLFOHJA
.

--
Regards,
Betty

---

Betty Fleming | [email protected] | Engineering Program Manager, Firefox
Mobile

[cpeterso]

Emilio on the Gecko Layout team is making good progress debugging the Gecko bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1551659#c11

[cpeterso]

The GV bug was fixed in GV Nightly last week:

https://bugzilla.mozilla.org/show_bug.cgi?id=1551659#c24

[cpeterso]

QA can't verify this fix until the "Open image in new tab" menu item is restored in #3487.

[ekager]

3487 has been completed and this should work in Nightly now

[softvision-miralobontiu]

Verified as fixed on the latest Nightly build 191002 (#12750607) with Samsung Galaxy Note 8 (Android 9), and Samsung Galaxy Tab A6 (Android 5.1.1).

[AndiAJ]

Reproducible on 2.2.0-rc.1 from 11/10 using a Google Pixel 2a (Android 9).

[cpeterso]

Reproducible on 2.2.0-rc.1 from 11/10 using a Google Pixel 2a (Android 9).

I'm not to reproduce the bug in Fenix Release (version Nightly 191011 13:35 from 10/11) or Nightly (version Nightly 191014 06:01 from 10/14) on my Moto G5 Plus (Android 8.1) using the same theverge.com page.

[AndiAJ]

Using a OnePlus 3A (Android 6.0.1) phone:
• Reprodubible on 2.2.0-rc.2
• Not reproducible on Nightly Build #12900612 from 17/10

[AndiAJ]

Reproducible on 2.3.0-rc.1 from 28/10 using a Samsung Galaxy Tab A (Android 9)

[cpeterso]

Not reproducible on Nightly Build #12900612 from 17/10

Reproducible on 2.3.0-rc.1 from 28/10 using a Samsung Galaxy Tab A (Android 9)

@AndiAJ - Does that mean this bug is not completely fixed and should be reopened?

[cadeyrn]

Does that mean this bug is not completely fixed

The GV bug was fixed in GV 71 and Fenix 2.3.0 still uses GV 70.