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

Setting new loop markers is complicated #5505

Closed
JohannesLorenz opened this issue May 17, 2020 · 4 comments
Closed

Setting new loop markers is complicated #5505

JohannesLorenz opened this issue May 17, 2020 · 4 comments
Labels
bug development branch This issue affects the development branch(master) gui
Milestone

Comments

@JohannesLorenz
Copy link
Contributor

Bug Summary

We have loop markers, e.g. in the song editor, to only play a part of a song in a loop. That part will be colored green. Originally, you could use middle and left click to comfortably select the markers.

There must have been some PR that changed the behavior on master. Now, only the right click works, and it always sets that marker that is closest to your mouse.

I find this not practical. I often have large/all parts of my song selected in the loop (projects saved like that) and then want to reduce the loop to the first or last few bars. Typically because the last bars are where I'm currently continuing the song. This gets difficult:

<------------------------ currently selected ------------------------------>
                                                wanted selection <-----> 

Setting the left marker of "wanted selection" requires me to scroll through half the song! (The right marker is easy, ofc)

Aside from that, there is no tooltip explaining how the new behavior is to be used. But I'm for switching back to the old behavior.

Steps to reproduce

See sketch above.

Expected behavior

Not being forced to scroll through half of your song.

Actual behavior

Scrolling through half of your song.

Affected LMMS versions

Current master. not on stable-1.2

@Spekular
Copy link
Member

Spekular commented May 17, 2020 via email

@JohannesLorenz
Copy link
Contributor Author

We could at least keep allowing the middle mouse key next to the new behavior, unless we plan to use it otherwise. That would at least help with the left marker.

Shift + LMB: Loop Start (snapping)
Shift + RMB: Loop End (snapping)

I like this even more. Tooltips would help to find out what the right shortcuts are.

@JohannesLorenz JohannesLorenz added this to the 1.3 milestone May 22, 2020
@Sawuare
Copy link
Member

Sawuare commented May 23, 2020

I changed the behavior in #4034.

@PhysSong PhysSong added development branch This issue affects the development branch(master) and removed master labels Aug 17, 2020
@PhysSong PhysSong changed the title Setting new loop markers is complicate Setting new loop markers is complicated Aug 17, 2020
@Rossmaxx
Copy link
Contributor

Can we close this one, since #6382 has been merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug development branch This issue affects the development branch(master) gui
Projects
None yet
Development

No branches or pull requests

5 participants