Skip to content

fix position of repeatDot in Emmentaler #28157

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

rettinghaus
Copy link
Contributor

refers to #28155

Brings not only a fix of the dot position but also a switch to open type. Regenerated the metadata.json file.

@rettinghaus rettinghaus marked this pull request as draft May 26, 2025 16:43
@rettinghaus rettinghaus marked this pull request as ready for review May 27, 2025 08:19
@rettinghaus
Copy link
Contributor Author

The TTF version looks slightly thicker than the OTF version.

@its-not-nice its-not-nice added the vtests This PR produces approved changes to vtest results label May 28, 2025
@Jojo-Schmitz
Copy link
Contributor

@rettinghaus please check the last commit if my backport Jojo-Schmitz#948, dealing with Emmentaler as an external font, I guess @worldwideweary and @mirabilos would appreciate...

Jojo-Schmitz pushed a commit to Jojo-Schmitz/MuseScore that referenced this pull request May 28, 2025
@rettinghaus
Copy link
Contributor Author

@Jojo-Schmitz I'm not so sure about this, because you could generate a newer ttf file from the source, so you'd never know which version is the "correct" one.

@Jojo-Schmitz
Copy link
Contributor

True, but if not this would maintain compatibility with the existing old font
And a new/changed font could well get converted to otf

@Jojo-Schmitz
Copy link
Contributor

Just rebased to resolve merge conflict or really changed something?
That code style error is back...

@rettinghaus
Copy link
Contributor Author

just a rebase

@rettinghaus
Copy link
Contributor Author

I reported the error in Ekmelos, so maybe we can get rid of this workaround completely at some point … 🤞🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vtests This PR produces approved changes to vtest results
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants