Mouse wheel shortcuts akin to plasma's media player widget

[AuraHerreroRuiz]

Something I'd like to see for this plasmoid is to add the actions the normal media player plasmoid has: Increase and decrease the current player's volume using the scroll wheel, and pause/resume on middle click.

[luisbocanegra]

Added in #117, please test it out

Update from the current version

git clone https://github.com/luisbocanegra/plasmusic-toolbar.git -b mouse-wheel-controls
cd plasmusic-toolbar && kpackagetool6 -t Plasma/Applet -u src

Then logout/reboot/restart plasmashell

Should the step size be configurable? I left it fixed to 5%

[AuraHerreroRuiz]

Added in #117, please test it out

Works perfectly! Thanks a lot for adding it.

Should the step size be configurable? I left it fixed to 5%

Hmm, from what I can tell, Plasma's default mediaplayer widget follows the volume step size specified in settings, just like the volume widget. (Input & Output > Sound > Configure Volume Controls...)

I feel it should follow that, if possible, for consistency with the rest of plasma.

[luisbocanegra]

Hmm, from what I can tell, Plasma's default mediaplayer widget follows the volume step size specified in settings, just like the volume widget. (Input & Output > Sound > Configure Volume Controls...)

Added, the cpp plugin from the default media player provides that https://github.com/KDE/plasma-workspace/tree/master/applets/mediacontroller/plugin (import org.kde.plasma.private.mediacontroller 1.0)

In theory, the default media player widget should always be shipped with Plasma, but since it is no longer a hard requirement to have working media shortcuts after MR applets/mediacontroller: port away from DataEngine (BUG 409190) I will add a conditional import and fallback to 5 if the module can't be imported.

[ccatterina]

Added, the cpp plugin from the default media player provides that https://github.com/KDE/plasma-workspace/tree/master/applets/mediacontroller/plugin (import org.kde.plasma.private.mediacontroller 1.0)

In theory, the default media player widget should always be shipped with Plasma, but since it is no longer a hard requirement to have working media shortcuts after MR applets/mediacontroller: port away from DataEngine (BUG 409190) I will add a conditional import and fallback to 5 if the module can't be imported.

@luisbocanegra @AuraHerreroRuiz I'm not very sure if it's worth doing this; I would prefer not to couple this widget too much with the media controller. For now, IMO it can remain a non-configurable value but if you guys prefer to make it configurarable we can add a new option in the settings.

[luisbocanegra]

I'm not very sure if it's worth doing this; I would prefer not to couple this widget too much with the media controller.

Understandable, reverted it back to 5%.

For now, IMO it can remain a non-configurable value but if you guys prefer to make it configurarable we can add a new option in the settings.

Personally, I never change the default on Plasma either, but for the people that do, an option that allows them to manually match the rest of the system could be nice IMHO.

[ccatterina]

Personally, I never change the default on Plasma either, but for the people that do, an option that allows them to manually match the rest of the system could be nice IMHO.

:+1: it's fine for me!