-
-
Notifications
You must be signed in to change notification settings - Fork 504
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
4.6 seems to break middle click (auto scroll) scrolling on Firefox #1649
Comments
Even if it took me a while to figure out that it's this add-on that's responsible, I can confirm that I'm also experiencing this issue. So I'll be keeping an eye on the thread. :) |
Reproducible here as well |
specific to firefox :D got it too. |
amazed about the activity here though
|
@ChristophSchmidpeter @debdrup So we have to try one by one https://github.com/code-charity/youtube/commits/424d609b14da9479f38db42795e474ca2ba75771 |
thanks, reporting to Firefox https://bugzilla.mozilla.org/show_bug.cgi?id=1827288 |
Firefox gave it a normal priority and said it might be fixed
in the meantime i can manually delete or overwrite it in the next firefox version ( @-moz-document url-prefix() { overflow-x: visible !important; } ) however the gain might outweight the loss, unless there is high correlation between ImprovedTube users in Firefox & Middle Click users (and Github Reporters)
( Such amazing correlation seems to be proven since you are nr 4. reporting @greenhouse95. |
Hey, so it breaks again on the latest version, assuming since the change you temporarily made was reverted. If it's too much maintenance to have middle click scrolling then it's okay, but it would be nice to have it working on Firefox. |
Yeah I don't feel like it will ever get fixed from either side (Firefox and Extension). So what I've done myself is use the Stylus extension to create a style, with this line: |
beautiful. |
It did work until it was removed on this commit. |
@ImprovedTube Can the removal of what's shown above be reverted? It's still broken |
hi @KSneijders! yes, you can try it https://github.com/code-charity/youtube/archive/refs/heads/master.zip (of course the most efficient would be a build script, just removing the html {overflow-x: hidden !important;} from the firefox version and already predicting until when that's good ) |
Hey @ImprovedTube, I tried it by loading the zip file as Temporary Extension in Firefox and the auto scroll works again! I didn't quite understand what you meant with the sentence "(of course the most efficient... ...until when that's good)". So I'll ask this just in case, will this change be released in the next update by any chance? 🙂 Thanks for the quick update and the plugin in general! ❤️ |
hi @KSneijders thanks
can be released, until firefox might also fix it.
(currently the gihub repo version is the same in all browsers, until we may use a build.py again to make the edits each browsers needs. yet running a line of CSS shouln't take relevant CPU time even for everybody in sum.) |
v4.443 moved the line here from general.css. That might just makes it run slightly earlier? youtube/js&css/extension/www.youtube.com/styles.css Lines 1 to 8 in 0e38a2a
|
The fix presented here works only for the first tab of Youtube, if I open another one it doesn't work there but keeps working on the first one. Closing all tabs and opening them again produces same thing. I'm pretty much clueless about how these things work so don't know if this is relevant, just pointing it out. |
@Onsekyl Yeah I got the same first tab issue, an improvement nonetheless. I would recommend looking for "Enhancer for Youtube", it's not on the Firefox Addons store anymore but you can find a zip online. It's not perfect but until this gets fixed it's a good patchwork solution. |
hi! @itsLantik @Onsekyl, confusing. you can try to remove the lines above from FireFox completely.
|
@ImprovedTube @itsLantik That worked! The scroll works now. To itsLantik: I recently moved from the Enchanger for Youtube as it began to malfunction too much. |
Hi, I noticed that if you press the middle mouse on top of the scrollbar the fast scroll works! |
hi! @Onsekyl thanks! (we can also fixing simply removing (if not fixing) the first few lines of our styles.css for firefox.) |
sorry this was missing html{ } also #2254 |
Bug Report:
⚬ BUG: Updating to 4.6 breaks autoscrolling, so pressing middle click and trying to scroll that way, on Firefox
⚬ HOW: Install 4.6 and try to middle click scroll on any (but especially video) page
⚬ EXPECTED (/preferred) behavior: For middle click scroll to work as expected
⚬ Context: This worked just fine on 4.0.17.1, however it also works on 4.4 so it might be a change between 4.4 and 4.6. On Chrome middle click scrolling works on 4.6 so this might be Firefox specific
Setup:
⚬ ImprovedTube Version: 4.6
⚬ Browser: Firefox 111.0.1
⚬ Settings: Happens on fresh install too
⚬ OS: Windows 11
I hope this isn't something specific to my setup, but I did try with a fresh portable version of Firefox and the addon
Edit2: if you navigate backwards from a video back to a channel you can middle click scroll for like half a second before it stops so it seems as soon as something loads it breaks
Edit3: it seems to always break most consistently when opening a video in a new tab
The text was updated successfully, but these errors were encountered: