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.

Sign up for GitHub

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

Jump to bottom

Specify bracketSpan in Verovio customisation #3851

Closed
rettinghaus opened this issue Nov 4, 2024 · 2 comments · Fixed by #3867
Closed

Specify bracketSpan in Verovio customisation #3851

rettinghaus opened this issue Nov 4, 2024 · 2 comments · Fixed by #3867
Labels
feature request

Comments

@rettinghaus
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Dealing with semi closed lists for attribute values is suboptimal. But using only the predefined ones in the ODD will lead to lost information while importing other formats.

Describe the solution you'd like
To handle bracketSpan@func values better in Verovio, we could add specific values to the list and make it a closed one in the Verovio customisation.

Additional context
@lpugin
Copy link
Contributor

lpugin commented Nov 5, 2024

I've been always hoping we could avoid to have a Verovio customization that adds things to the MEI schema. However, in that case it might be acceptable since the generated MEI would still be valid with the official schema, wouldn't it?

@rettinghaus
Copy link
Contributor Author

Yes, as full MEI has specified an open list, basically every value is valid. And we wouldn't mess with the MEI structure, only specify the values we want to generate with libmei for Verovio.

@rettinghaus rettinghaus mentioned this issue Nov 25, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Specify bracketSpan in Verovio customisation rettinghaus/verovio
2 participants
@lpugin @rettinghaus