-
Notifications
You must be signed in to change notification settings - Fork 519
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RFP: zoom per eTLD+1 per tab not remembered on some sites #1636
Comments
One more thing: If you set default zoom to anything other than 100% in |
RFP ignores site settings for zoom preferences on each new tab, and any eTLD+1 change. It's not meant to "reset" if you stay on the same eTLD+1 in the same tab using http://www.hdtvsolutions.com/ nothing
just ETP strict
just RFP
won;t check the 4th combo of ETP + RFP AFAICT this isn't anything to do with ETP, it's RFP and how it's handling the eTLD+1 for some sites ? |
same with https://www.chicagomanualofstyle.org/ - make sure you use a same eTLD+1, e.g the red box - it's RFP on it's own FWIW, I changed to ETP standard with RFP and same result |
STR
I wonder if it's the
Just tested tor browser 12.0.3 based on ESR102 and using http://www.hdtvsolutions.com/ the problem does not exist regression ended with
|
@practik , see my last comment, last line. I think you're right, this has something to so with ETP (wasn't that switched on by default in FF94?). I bisected my FF portables, which are all defaults except I sanitize on close and a few visual tweaks. 93 was good, 94 was bad But I ran moz-regression on the whole 94 nightly cycle and came up empty |
must be something that was backported/flipped - regression is in 97
|
what made you think it might be ETP, the TB test? I run mozregression as well but ETP doesn't seem to make a difference. I also tried on https://www.youtube.com and the |
just speculation .. you know, urlclassifiers and stuff and TB doesn't use any of ETP (AFAICK). I tried to look at commits on mercurial for 2021-12-16 but not sure how to get an actual list. There were a couple of interesting patches but TBH, IDK what I'm looking for |
understood. did you fill a ticket at bugzilla or we can avoid that since you tagged Tom above?
you could run |
we'll see if he pokes his head up, otherwise I might poke him in a week's time |
Has this ever worked? The code seems to imply this is what should happen and it's working as 'intended' and has always done that. |
Yes. Issue was not present before - hence moz-regression good vs bad |
Just to confirm this: I use the Chicago site pretty regularly and started noticing the issue there about two weeks ago when I updated to FF 109. It never used to happen before that. |
🟥 https://github.com/arkenfox/user.js/wiki/5.2-Troubleshooting
maywill be closed as invalid🟪 REQUIRED INFO
a. In a fresh profile, flip
browser.contentblocking.category
tostrict
andprivacy.resistFingerprinting
totrue
.b. Go to http://www.hdtvsolutions.com/ and change the zoom to something other than 100%.
c. Go to any other page on that site.
The text was updated successfully, but these errors were encountered: