-
-
Notifications
You must be signed in to change notification settings - Fork 335
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
Video rotates from landscape to portrait Firefox Android #1993
Comments
+1 came to report the exact same behaviour, as well as doing so randomly and during skips, seems to sometimes react to touch input to the screen Version Info:
|
I guess this is the problem I keep having on Android Firefox, as well. |
Update:Based on the time that I think this bug started happening, I found the firefox releases around that time and after trying v122 of Firefox android with sponsor block, the issue is not present. Must've been a change in Firefox Android 122.0.1+ that is causing this In case anyone wants to rush using an outdated version to continue using sponsor block without this issue: |
I'm also experiencing this issue. |
As a semi-workaround, this still happens if you enable auto-rotation, but you won't notice it mostly. The exception is if your device is flat on a table or something where the orientation isn't clear (in which case picking it up will fix). Annoying, but mostly does the job if you can handle having autorotation switched on (I found I mostly can, though can cause the odd issue using phone in bed). Still be nice if it was fixed though given it used to just work properly...might be why few people seem to have noticed though. And while I'm here: still happening in Firefox 130. |
Also, as it was a update from Firefox that is causing this, you can at your
own discretion use an older version.
https://github.com/mozilla-mobile/firefox-android/releases/tag/fenix-v122.0.1
for example does not have this issue
…On Sat, 14 Sept 2024, 23:44 caeci11ius, ***@***.***> wrote:
As a semi-workaround, this still happens if you enable auto-rotation, but
you won't notice it mostly. The exception is if your device is flat on a
table or something where the orientation isn't clear (in which case picking
it up will fix).
Annoying, but mostly does the job if you can handle having autorotation
switched on (I found I mostly can, though can cause the odd issue using
phone in bed). Still be nice if it was fixed though given it used to just
work properly...might be why few people seem to have noticed though.
—
Reply to this email directly, view it on GitHub
<#1993 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABJVVQUBXSGRPZUOYMTTSYTZWSU4BAVCNFSM6AAAAABFRAFDO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJRGE3DQMJRGE>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Tap forward or backward to move 10s, then the other way to come back where you left. Sidenote, outdated versions run the risk of unpatched security flaws, it is not recommended to use them. |
I would have to destroy my Firefox profile to downgrade, since Android no longer allows downgrading applications over an old installation :( |
This is true, I personally however have multiple different versions of
Firefox on my phone for different uses- Firefox beta, 'main' old version
(security risks associated acknowledged and accounted for (gotta love Open
Source)) and also Firefox Klar/Focus. Having a Mozilla account to sync
everything fixes most issues, tho some parts of configuration do need to be
redone- I've attempted to have a look to see the diff in Firefox builds
that broke this extension, but to no success as of yet.
If I were to guess it could either be an issue with Firefox itself that was
fixed, but the extension had an underlying reliance on- if I find the cause
I'll open a PR either here, or on the FF pages
…On Mon, 16 Sept 2024, 10:43 Catherine, ***@***.***> wrote:
I would have to destroy my Firefox profile to downgrade, since Android no
longer allows downgrading applications over an old installation :(
—
Reply to this email directly, view it on GitHub
<#1993 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABJVVQRIBBTEALDWQ2CLWJTZW2K27AVCNFSM6AAAAABFRAFDO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJSGMZDIOJYGU>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Thanks for looking into the issue! Much appreciated. |
Same issue here. I dont have any extensions. v133.0 Happens randomly over time or when tapping the video in fullscreen (to show time bar or to fast forward / back). |
It bothers me so much... I've been sick the last few weeks and youtube was kind of the only thing I had energy for, and the endless rotation bug was such an annoyance. |
Workaround available
If you use Firefox as your daily driver (which you probably do as you are here in this thread) use Fennec 122.0.0 instead: Make sure to set your stores to not update the app and do not use this browser for anything different than YouTube to prevent from security issues. |
Surely using your phone with auto rotate enabled (which MOSTLY resolves this, providing your phone has clear orientation and isn't flat on a table) is a better workaround than using a long outdated version of a web browser. Don't get me wrong, this issue really sucks. But don't use a really old browser and lose every security fix that's been published in months. Frankly, disabling sponsorblock entirely and living with ads would be a better solution than that. |
Thanks much! Another option that I discovered recently is PipePipe (a NewPipe fork) which seems to work well for long-form videos, sans a few issues with subtitles. I will probably stick to the browser personally, but others who look up this thread might want to consider PipePipe instead.
I end up lying in bed in one of the few ways that hurt somewhat less than the rest, and the phone ends up in whichever orientation it gets.
Risk acceptance is a perfectly reasonable option. (I would ask you to describe the threat model you're considering here, but that would be both pointless and off-topic.) |
I dont have sponsor block yet still get the issue. I have no extensions. |
I can't seem to reproduce this without extensions. At the moment am looking at this change to FF: Mercurial / GitHub Note that also seems to be not only sponsorblock that experiences this - have found a couple other extensions that can cause this on their own too, all extensions that attempt to add features to Fullscreen video (ImprovedTube for example) As a side note, indeed risk acceptance is an option, I decided to stop using an older FF build as of this CVE however: https://www.mozilla.org/en-US/security/advisories/mfsa2024-51/ Even with the old build of FF being used for only YouTube (a bit like a PWA) with other domains blocked this RCE CVE could provide a serious risk of the extensions themselves having unauthorized code exec. -somewhat offtopic but thought would add this |
(If someone successfully compromises SponsorBlock or uBlock Origin, the only two extensions I use, via Firefox Store, in order to get code exec on my Android phone specifically I will take it as an exceptional degree of praise. It's such a staggeringly unlikely scenario that I don't think it's worth considering.) |
There's a Firefox bug causing this on some phones even if you have no extensions: https://bugzilla.mozilla.org/show_bug.cgi?id=1927557 |
Thanks. I now backlinked this issue here in the Firefox tracker. |
I believe the cause may be rooted in the background script being unloaded/removed, am currently testing with this change to help confirm, which just sets the background script to be persistent. Not ideal in the long term, however for now running this artifact from local install on FF nightly and having had around 4hrs of videos played- the rotation issue is gone. |
Correction: Not when it is removed, only when the runtime reloads it back in. So when the background script (or 'event page' as it is referred to when persistence is false) is not running and an event (interacting with video controls, segment updates, or a segment skip etc) is called that requires a message to be handled and the script becomes active - this bug occurs. Have experimented with narrowing down what about loafing the script causes this but haven't had much success- the event listeners that make up most of the script seem to all follow mdn web docs recommendations and practice, could be elsewhere in the utils or even node/react -Dom etc but I am not familiar enough with these to locate it. Could even be a FF issue more generally. My reccomendation would be to simply set persistence to true as this mitigates the issue until more info, or there is an update for https://bugzilla.mozilla.org/show_bug.cgi?id=1927557. |
Mobile is where a non-persistence background script is very important. Running the background script even when YouTube is not open, and you are just browsing other sites, is not a good idea. I think this needs to be resolved in Firefox itself |
@Wolfdv1 Thanks for your PR! I've had to sign it myself because apparently there's no way to load an unsigned XPI on Andriod, but it works fine. It does seem to not change the color of the progress bar for some reason, but I also don't really care about that; it skips the sponsored segments and I'm happy with it. |
@whitequark No problem, another lil workaround until it is fixed in Firefox itself. It is almost certainly purely a bug on Mozillas end- did a test using one of their own example extensions as can be seen here, and it occurs also. Will update the bugzilla thread/report as needed. Regarding the workaround, as Ajayyy said- having the background page be persistent does mean it will be loaded on all sites, the code seems trustworthy but still could have the theoretical potential risk to open up CORS vector issues. Fun fact also: On Firefox Nightly for android you can go to about:config and set ,,xpinstall.signatures.required" to false, and then go into Settings>About Firefox Nightly>Click on the logo 5x and then in settings under advanced 'Install add-on from local file' will allow an unsigned local zip to be permanently installed :) ofc know the potential risks etc etc and do at your own risk, but wanted to add thisAnyway, hope the FF Devs can fix this asap. I understand why Ajayyy wouldnt want the bg script to be persistent for all users (hence closed the PR after seeing the reply here) - btw thank you for all the hard work! |
This was working really nicely on the Android version of Firefox until about a week ago. Since then, it's been causing the video player to rotate back to portrait periodically during playback (sometimes seems associated with skips, but not always). Happens within a few minutes regardless.
Phone is configured with automatic rotation disabled. Commencing in portrait, Firefox will rotate to landscape when fullscreen playback is selected. After a few minutes of playback with sponsorblock running, the video will rotate back to portrait. Skipping forward/back using double tap restores landscape orientation, as does exiting and re-entering fullscreen. Disabling plugin fixes the issue.
Plugin version 5.5.9
Firefox version 124.1.0
Pixel 7 Pro 5 March 2024 firmware
The text was updated successfully, but these errors were encountered: