I added the ability to set the scrolling speed when starting continuous scrolling.

I found a bug that should be resolved before merge.

Actually, the "issue" seems to happen without my pull request. Is it intended for some reason for using the continuous scrolling action twice in the same direction to stop the scrolling completely?

Yes that is on purpose. For people using a foot switch, noise or other to toggle the scrolling on and off.

@knausj85 Will test this. If someone else has time before that please go ahead.

Actually, the "issue" seems to happen without my pull request. Is it intended for some reason for using the continuous scrolling action twice in the same direction to stop the scrolling completely?

I initially thought this was a bug too; these commands used to be compatible with repeaters and such. I guess I missed that memo!

If needed, we can explore how to make these more friendly for hotkeys / pedals in a separate issue

I tested this a bit and it's a great improvement. I've a few minor suggestions before we merge.

Actually, the "issue" seems to happen without my pull request. Is it intended for some reason for using the continuous scrolling action twice in the same direction to stop the scrolling completely?

There is a separate few bugs with this logic though that we might as well fix while we're here:

• the imgui should probably close & tag be cleared when this happens.

I took a quick swing at fixing that.

I identified an issue where if the scrolling speed becomes less than one but non-zero the result is no scrolling. Should I address that by rounding up? (This can happen when trying to set the scrolling speed to a small number)

It looks like float values for the scrolling speed get truncated, so changing the scrolling speed from some values to certain other values through voice commands has no effect. I am not sure if we should try to address that to make increasing speed value always increase the actual scrolling speed, but the issue is worth noting.