mozilla / contain-facebook

Facebook Container isolates your Facebook activity from the rest of your web activity in order to prevent Facebook from tracking you outside of the Facebook website via third party cookies.
Mozilla Public License 2.0
979 stars 176 forks source link

Fastmail "show details" broken #886

Open eam opened 2 years ago

eam commented 2 years ago

Actual behavior

Clicking on the "show details" link in the Fastmail webui does not show details (expanding From/To/Cc/Subject/etc beyond the first line).

Expected behavior

Clicking this link should expand the details in the fastmail webui

Steps to reproduce

  1. Enable Facebook Container
  2. Open a new tab and read an email on Fastmail
  3. Click the "show details" link

Notes

I have been using Facebook Container on this platform for a long time without issue. I am certain a recent update caused this issue - within the last month or two.

joshbode commented 2 years ago

I have also have this exact issue (it also affects the folders drawer), which I've seen sporadically in the past (I even raised an issue with Fastmail support, but closed it when the issue went away). Note - this issue occurs in the Fastmail beta site (betaapp.fastmail.com) also.

It took ages to diagnose because even though I suspected an add-on (e.g. uBlock, Privacy Badger, etc), I never disabled this add-on, because... why would it have an effect on fastmail.com? :)

joshbode commented 2 years ago

It may be related to this CSS rule: https://github.com/mozilla/contain-facebook/blob/9ab4eb7c2f8ad535c0f171400903140083ab156f/src/content_script.css#L352

which matches the class is-hidden on the div for the sidebar:

<div id="v183" class="v-Split v-Split--left is-hidden" style="position: absolute; z-index: 1; transform: translate3d(-220px, 0px, 0px); inset: 0px auto auto 0px; width: 200px; height: 100%; will-change: auto;">

similarly for the "show details" button:

<div aria-hidden="true" class="v-Message-details is-hidden" style="height: 0px;">
joshbode commented 2 years ago

yep - the display: none style is definitely coming from content_script.css:

image

neilj commented 2 years ago

(I'm a developer at Fastmail.) It looks like this extension is also causing the message list to be scrolled to the top whenever you open an email. Why is this extension modifying anything on our site at all? Shouldn't it only be applying to Facebook's domains?

joshbode commented 2 years ago

@neilj #896 appears to fix this behaviour, too (I've just tested this in Firefox Developer Edition)

I suspect this extension applies to all pages indiscriminately because in theory, Facebook tracking can be embedded in any page (but of course, never Fastmail!) and I'm not sure there's a way to apply (or not apply) CSS to specific pages

maxxcrawford commented 1 year ago

@joshbode @neilj We reviewed the PR and merged it in. It will be a part of our next release!

Shout out @joshbode for the PR.

(Note - No exact word on immediate timing for that release, but I can estimate in the next few weeks)

joshbode commented 1 year ago

thanks @maxxcrawford :)

dcelasun commented 1 year ago

@maxxcrawford can you share a build here in the meantime? With signed extensions, it's no longer trivial to make local changes and the only alternative is to disable the extension. Neither sound great if the update is weeks away.

krystian3w commented 1 year ago

[!important]

Update addon to 2.3.10 or 2.3.11.


no longer needed after update addon to 2 3.10+ Maybe as fallback works: ```css .is-hidden:not([id^="fbc-"]) { display: revert !important; } /* !: but still break layout if webmaster enforce too low other value as initial/inherit. */ /* !: also no possible in that way detect new release new XPI */ ``` but to works need Stylus or: uBlock Origin/AdGuard (not "Lite => MV3" if you can not locally edit rules). ```adb ! AdGuard #$#.is-hidden:not([id^="fbc-"]) { display: revert !important; } ! uBO *#$#.is-hidden:not([id^="fbc-"]) { display: revert !important; } ```
krystian3w commented 1 year ago

5 months later: https://github.com/mozilla/contain-facebook/issues/898#issuecomment-1507446045

joshbode commented 1 year ago

Hi @maxxcrawford - is there anything we can do to help prepare for a release (testing a pre-release, for example)?

joshbode commented 1 year ago

Also, we can probably close this particular issue, because Fastmail has already worked around the issue.

krystian3w commented 8 months ago

I see someone pushed two new releases with fix for .is-hidden issue:

https://addons.mozilla.org/addon/facebook-container/versions/