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

newest 4.3 build mousewheel scrolling doesn't work when scrolling down or up when not on video #1626

Closed
jeremy-hill opened this issue Mar 19, 2023 · 7 comments
Labels
Feature request Wish or idea help wanted Just an old github standard we add automatically. (The team can remove it when working on it.)

Comments

@jeremy-hill
Copy link

Bug Report:

BUG: When using this addon, mousewheel when mouseovering the video adjusts volume by default, which still works. However, mousewheeling ANYWHERE in the window adjusts volume now, even when not mouseovering the video.

HOW: Start a video, move your mouse anywhere where the video is not, and mouseover. Volume will adjust, the page will not scroll.

EXPECTED (/preferred) behavior: The page scrolls.

/ ⚬ Context: Doesn't matter if you're in Default view or Theater view.

Setup:

⚬ ImprovedTube Version: 4.3
⚬ Browser: Google Chrome Version 111.0.5563.65 (Official Build) (64-bit)
⚬ Settings: (I'm not really comfortable with this; a lot of my watch history is in here.)
⚬ OS: Windows 10
⚬ Device: PC

@jeremy-hill jeremy-hill added Bug Bug or required update after YouTube changes help wanted Just an old github standard we add automatically. (The team can remove it when working on it.) labels Mar 19, 2023
@ImprovedTube
Copy link
Member

hi! @jeremy-hill thats a nice feature. Funny thing: we don't have it (yet). Our extension does nothing by default, besides the 3 three extra buttons.

@ImprovedTube ImprovedTube added Feature request Wish or idea and removed Bug Bug or required update after YouTube changes labels Mar 19, 2023
@jeremy-hill
Copy link
Author

Gosh maybe I'm on the wrong github! This is for *Improve YouTube!*🎧(for YouTube + Video), extension ID bnomihfieiccainjcjblhegjgglakjdd?

If so, I'm saying this settings:
image

applies everywhere on the page, not just when over the video -- you can't scroll down at all. Please let me know if I'm on the wrong github though!

@jeremy-hill
Copy link
Author

Yeah I think I have the right place, as these guys (not me), who posted in the Chrome Web Store are also having the issue:
image

@Aweron
Copy link

Aweron commented Mar 19, 2023

I actually came here to ask about this bug/change as well. I believe I setup this feature manually after switching from a different extension that had that feature, and I assume @jeremy-hill might have as well a while ago, so they forgot that and didn't mention it. I think it's just a manual hotkey setting for volume. I would really like if it could be made how it was before, as it was practically the only reason I used the extension.

ImprovedTube added a commit that referenced this issue Mar 20, 2023
@ImprovedTube
Copy link
Member

hi! @jeremy-hill ohhh 😆 sorry for my confusion! My bad

also,

theater
would you prefer the scroll-overwriting to happen trigger only on the video, not on the player background?

(then we are prepared just replacing the elements.player with elements.video here) (@MohamedLebda)
https://github.com/code-for-charity/ImprovedTube-for-YouTube/blob/a1fc43675c6842ee5dcb4a1a22981af60f6d5d73/content-scripts/website-context/youtube-features/shortcuts.js#L135-L145
(just a question of GUI.

we can get back the specific feature button named "scroll for volume, while mouse is on the video", as it was in old version 1.x ( @MohamedLebda )

(next to mouse over as a building block for any shortcut maybe 🤔)

@jovibor
Copy link

jovibor commented Mar 20, 2023

However, mousewheeling ANYWHERE in the window adjusts volume now, even when not mouseovering the video.

Confirmed. I have exactly same bug.

@ImprovedTube
Copy link
Member

Hi @jovibor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request Wish or idea help wanted Just an old github standard we add automatically. (The team can remove it when working on it.)
Projects
None yet
Development

No branches or pull requests

4 participants