[WIP] resume other media on pause#631
Conversation
|
@jh30uk you can test out the build by uninstalling fluent flyout, and installing the version here (edit, wrong link) |
|
actually im not sure if you have to uninstall fluent flyout, just know that it might cause problems if you have both running, be careful |
|
https://github.com/nopeless/FluentFlyout/actions/runs/23824185599 |
|
It works 100% fine, and it is even more reliable then before for pausing the media. |
|
@jh30uk ok, can you get back on this after you have used for a few days? to make sure there are no bugs. once you confirm, @unchihugo can make the settings ui for that |
|
It works fine and the odd glitch of not pausing the media since day one was on my daily Windows install (it is in need of a wipe as getting glitchy) but on my test Fresh/bare min USB install it works 99.9% of the time and therefor the new resume also works. I assume v2.10.1 will not include this function for now? |
|
Bump! |
|
Will get to this once I'm free! I'm thinking of moving this feature to Advanced Settings, since it's not necessary related to the Media Flyout (where it currently is). Are you guys fine with that? |
|
Yea one small thing, can we get a a delay as it may piss some peeps off as if you use the mouse cursor to skip forward using the timeline in YouTube the other previous audio will resume until you click the mouse again (so if you are clicking the timeline multiple times to find a certain spot its a bit of a PIA). It does not do this if you use the cursor keys to skip forward on the timeline. A setting to let us choose between 0-5secs would fix this IMO. |
I don't see myself using the feature, but it seems appropriate
@jh30uk nice catch. this is why you don't really roll out features fast, like, I never could have thought of that happening. I'll look into this for a bit |
|
I looked into the api and there isn't an explicit "is scrubbing", but there is a "change position" event, meaning that we can kinda guess what the user is doing by checking their scrubbing state. Does this seem like a good idea? |
|
Sure and I will gladly test it for you. |
|
@jh30uk @unchihugo while implementing the feature, I found a bug?, the resume feature just completely messes up user experience. I need a resolution for this #675 |
#144
Currently, no configuration is needed. If UX is ok, supporting UI config should be introduced to this branch