Skip to content
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

Pause video bug #2225

Closed
0x0017 opened this issue Apr 30, 2024 · 7 comments
Closed

Pause video bug #2225

0x0017 opened this issue Apr 30, 2024 · 7 comments
Labels
Bug Bug or required update after YouTube changes Completion / Revision Rethink, complete, improve, tweak our feature or structure. Exemplary! worth reading & repeating. good first issue A GitHub standard for inviting (new) contributors *Congratulations in advance!* help wanted Just an old github standard we add automatically. (The team can remove it when working on it.) up-for-grabs (a github standard for inviting new contributors) - Welcome! ♥ Users-Documentation / HowTo We should repurse (some of) this thread to introduce users

Comments

@0x0017
Copy link

0x0017 commented Apr 30, 2024

Bug:
While the “Disable video playback on hover” and “Pause while I watch a 2nd video” options are enabled, a bug occurs when hovering over a video preview on the search page, the 1st video stops playing, although it doesn't happen on the main page and a channels videos page.
Persist on both private and default modes.
Browser: Firefox 125.0.2 | Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:125.0)
OS: Windows 10
.webm
improvedtube.json

@0x0017 0x0017 added Bug Bug or required update after YouTube changes good first issue A GitHub standard for inviting (new) contributors *Congratulations in advance!* help wanted Just an old github standard we add automatically. (The team can remove it when working on it.) up-for-grabs (a github standard for inviting new contributors) - Welcome! ♥ labels Apr 30, 2024
@ImprovedTube ImprovedTube added the Completion / Revision Rethink, complete, improve, tweak our feature or structure. label Apr 30, 2024
@ImprovedTube ImprovedTube added this to the Little or easy(?) milestone Apr 30, 2024
@ImprovedTube ImprovedTube added Exemplary! worth reading & repeating. Users-Documentation / HowTo We should repurse (some of) this thread to introduce users labels Apr 30, 2024
@ImprovedTube
Copy link
Member

ImprovedTube commented Apr 30, 2024

hi! @0x0017 that's (such) a (very) nice bug report!
(and i might be slightly biased to say so because) after reading "search page" specifically,
i have the feeling it is an easy fix and the code is just missing that detail, also since it is relatively new.

Once we are at it, can we define what else to improve or add? I guess:

  • “Pause while I watch a 2nd video” should also have a sub-option:
    • enabled by default "don't count previews on hover"

@0x0017
Copy link
Author

0x0017 commented May 1, 2024

Glad to help. Now that's a quick response!
Yes having that option would be very handy. It's not quite clear to me how anyone could want this behavior without this option enabled by default, but it's good that you don't exclude this possibility and let the user choose.
I've had one more bug while using the addon, so I'll try to make another report, but I don't think it will be easy to fix.
Thanks again. I am glad to see that the addon is actively developing and there are more and more customization possibilities

@raszpl
Copy link
Contributor

raszpl commented Jun 16, 2024

two bugs here

1 search page hover, fixed by #2383

2 reported in #2305 autoplayDisable() doesnt filter out hover previews no wait, autoplayDisable() doesnt pause those because it only works either on '/watch?' or channel page. I never saw those video previews on watch/channel pages

@raszpl
Copy link
Contributor

raszpl commented Jun 16, 2024

done

@0x0017
Copy link
Author

0x0017 commented Jun 25, 2024

Thanks! I'll check the fix when Firefox update add-on on their site. For now "Check for updates" can't find new version (still on 4.937)

@raszpl
Copy link
Contributor

raszpl commented Jun 25, 2024

You can try manual patch part of https://github.com/code-charity/youtube/pull/2356/files#diff-5d5fcd053d89ea6d0186c65cc18b0f9d5a3f6fc9ee3a179c9065dd78017e0667

nah, I tried locally and cant make modified firefox extension package work :(

@0x0017
Copy link
Author

0x0017 commented Jun 27, 2024

It works perfectly now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Bug or required update after YouTube changes Completion / Revision Rethink, complete, improve, tweak our feature or structure. Exemplary! worth reading & repeating. good first issue A GitHub standard for inviting (new) contributors *Congratulations in advance!* help wanted Just an old github standard we add automatically. (The team can remove it when working on it.) up-for-grabs (a github standard for inviting new contributors) - Welcome! ♥ Users-Documentation / HowTo We should repurse (some of) this thread to introduce users
Projects
Status: Done
Development

No branches or pull requests

3 participants