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

1.3 Alpha playhead behavior bug in pianoroll #5962

Closed
Feihei opened this issue Mar 23, 2021 · 7 comments · Fixed by #5965
Closed

1.3 Alpha playhead behavior bug in pianoroll #5962

Feihei opened this issue Mar 23, 2021 · 7 comments · Fixed by #5965
Assignees
Labels

Comments

@Feihei
Copy link

Feihei commented Mar 23, 2021

When I pan the pianoroll window horizonally, the playhead line will move too, and then it will not align with the triangle cursor on top. I am using lmms-1.3.0-alpha.1.33+g4f74151f0-mingw-win64 version in win10

@Veratil Veratil self-assigned this Mar 23, 2021
@Veratil
Copy link
Contributor

Veratil commented Mar 23, 2021

Hi Feihei, I cannot reproduce this bug. Can you tell me if there was anything you did beforehand?

@qnebra
Copy link

qnebra commented Mar 23, 2021

https://youtu.be/JFFEb6qVE1U

Windows 10, latest alpha version, occurs on zoom levels above 100%

@Veratil
Copy link
Contributor

Veratil commented Mar 23, 2021

Got it! Thanks!

@Veratil
Copy link
Contributor

Veratil commented Mar 26, 2021

@Feihei @qnebra can you test the build in #5965 to try and reproduce the bug? All my testing trying to reproduce or even produce other odd bugs related to the position line seem to be squashed. :)

@Feihei
Copy link
Author

Feihei commented Mar 26, 2021

@Feihei @qnebra can you test the build in #5965 to try and reproduce the bug? All my testing trying to reproduce or even produce other odd bugs related to the position line seem to be squashed. :)
Thank you for the reply.
but sorry I don't know how to test, I don't have experience in pulling or building the source by myself

@qnebra
Copy link

qnebra commented Mar 26, 2021

There is lmms build bot comment in mentioned PR. You can download build to test pull request.

@Feihei Feihei closed this as completed Apr 2, 2021
@qnebra
Copy link

qnebra commented Apr 2, 2021

Please close it after fix is merged to current master branch. But also this is your decision, so it is fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants