Closed Bug 1790154 Opened 2 years ago Closed 2 years ago

fandom.com page layout problems in Firefox for Android with ETP Strict enabled

Categories

(Core :: Privacy: Anti-Tracking, defect, P3)

All
Android
defect

Tracking

()

RESOLVED FIXED
Tracking Status
firefox105 --- fixed
firefox106 --- unaffected
firefox107 --- unaffected

People

(Reporter: u713120, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

Possible JavaScript CSS style injection failure on main.bundle.js, affected CSS class ".ad-slot-placeholder"

Thanks for the bug report. I can reproduce the layout problem on https://lotr.fandom.com/wiki/Main_Page in ETP Strict mode.

If I switch to ETP Standard mode or disable ETP, the layout problem goes away, but that's because the "JOIN A PASSIONATE COMMUNITY OF PEOPLE WHO LOVE WHAT YOU LOVE" and "JOIN FAN LAB, AN EXCLUSIVE ONLINE COMMUNITY OF FANS" messages disappear. It seems like those messages are placeholders shown when ads are blocked.

I don't see those messages in Firefox iOS, with or without ETP Strict mode.

Blocks: tp-breakage
Severity: -- → S4
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P3
Summary: Basic website layout broken in Firefox for Android → fandom.com page layout problems in Firefox for Android with ETP Strict enabled

Hello Chris, thank you for the feedback.

I see, so it seems to be triggered by ETP strict mode enabled. I tested for the bug as well on Brave Browser for Android with all privacy shields on strict mode but I was not able to reproduce the error (Brave Browser 1.43.90; Android 12)

Adittionally I was able to reproduce the error on desktop Firefox 104.2 using DevTools mobile device emulation feature.

Since Firefox for iOS uses webkit, do you think there could be any feature detection script affecting the Gecko engine?

Status: NEW → RESOLVED
Closed: 2 years ago
Flags: needinfo?(cpeterson)
Resolution: --- → WORKSFORME
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---

Sorry I was logged from the phone, I don't know if I was the one who marked the bug as resolved by mistake when submitting a comment so I reopened it just in case

(In reply to anotheranonymoususer from comment #2)

I see, so it seems to be triggered by ETP strict mode enabled. I tested for the bug as well on Brave Browser for Android with all privacy shields on strict mode but I was not able to reproduce the error (Brave Browser 1.43.90; Android 12)

Me too. I wasn't able to reproduce the problem with Brave or the DuckDuckGo browser, even though they block ads on the page. I think Firefox's ETP Strict mode is blocking more elements (than Brave, DuckDuckGo, or ETP Standard mode) and those missing elements are causing the page layout problem.

Since Firefox for iOS uses webkit, do you think there could be any feature detection script affecting the Gecko engine?

That's probably the case. Since Firefox for iOS and Android use different engines, they probably implement ETP differently and maybe Firefox for iOS's implementation doesn't block the same elements as Android's ETP Strict mode.

Flags: needinfo?(cpeterson)

I see, thank you very much

Since this bug appears to be a webcompat problem with ETP Strict mode, I'm moving this bug to the Anti-tracking component.

Severity: S4 → --
Component: General → Privacy: Anti-Tracking
Priority: P3 → --
Product: Fenix → Core
Severity: -- → S3
Priority: -- → P3

Issue persists on Firefox 105.1.0

Issue appears to be fixed on firefox 105.2

Chris, can you please confirm this? Feel free to mark as resolved if confirmed

Flags: needinfo?(cpeterson)

(In reply to anotheranonymoususer from comment #9)

Issue appears to be fixed on firefox 105.2

Chris, can you please confirm this? Feel free to mark as resolved if confirmed

Looks good to me in 105.2. I assume recent changes to the Strict ETP block list fixed this.

Thanks for reporting this bug and verifying it's been fixed!

Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Flags: needinfo?(cpeterson)
Resolution: --- → FIXED

(In reply to Chris Peterson [:cpeterson] from comment #10)
Thanks to you, Chris and Tim

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: