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

Staves are in the wrong order when there are two columns #1300

Closed
JoyfulGen opened this issue Feb 20, 2025 · 3 comments
Closed

Staves are in the wrong order when there are two columns #1300

JoyfulGen opened this issue Feb 20, 2025 · 3 comments
Labels
bug Something isn't working Severity: MEDIUM Moderate problem affecting usability or a necessary feature. to-be-merged Fixed, but waiting for RISM to merge changes to develop.

Comments

@JoyfulGen
Copy link
Contributor

If I create two columns of staves in Neon, the staves remain ordered as if there was only one column. So rather than the order being one column, then the second, it goes left-right-left-right-etc. This is easiest to demonstrate:

In the screen recording, you can first see that I have two columns: orange and blue. In the staff highlight, you can already see that the colours are wrong. To illustrate that, I move from bounding box to bounding box using the right arrow, to show that the order is jumping between columns.

Neon.columns.weird.mov
@JoyfulGen JoyfulGen added bug Something isn't working Severity: MEDIUM Moderate problem affecting usability or a necessary feature. labels Feb 20, 2025
@yinanazhou
Copy link
Member

yinanazhou commented Feb 25, 2025

Update:

When first loaded to Neon, the staff highlight order is correct.

Image

Image

But as soon as a change is made on staff level, the highlight order is wrong:

Image

yinanazhou added a commit to DDMAL/verovio that referenced this issue Feb 25, 2025
@yinanazhou yinanazhou added the to-be-merged Fixed, but waiting for RISM to merge changes to develop. label Feb 25, 2025
@yinanazhou
Copy link
Member

Hii @JoyfulGen, the changes have been merged. This should be fixed now :)

@JoyfulGen
Copy link
Contributor Author

@yinanazhou this is indeed fixed! I'm sorry this took a while to test--I was waiting to have a proper two-column file, and I finally do, and it looks great! I'll close the issue now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Severity: MEDIUM Moderate problem affecting usability or a necessary feature. to-be-merged Fixed, but waiting for RISM to merge changes to develop.
Projects
None yet
Development

No branches or pull requests

2 participants