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

Added the possibility to use straight flags #9307

Merged
merged 2 commits into from
Sep 28, 2021

Conversation

cbjeukendrup
Copy link
Contributor

StraightNoteFlags.mov

Note: currently, it is broken when using Leland, because the metadata file for Leland needs to be fixed, but @oktophonie is working on that.

For fonts that do not support straight flags, Bravura is used as a fallback. (We may choose to implement a fallback chain, so that MuseScore first tries the font the user specified, then Leland, then Bravura.)

@@ -6015,6 +6032,23 @@ constexpr const std::array<const char*, size_t(SymId::lastSym) + 1> SymNames::s_
"Large brace",
"Larger brace",

"Combining flag 8 (1024th) below straight",
Copy link
Contributor

@Jojo-Schmitz Jojo-Schmitz Sep 28, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Translatability of those would be needed, now that they became '1st class citizen'

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why would that be needed? As far as I know, these names are never exposed in the UI. The only reason that I added them, is to make sure that the lists of names stay consistent.

By the way, the names of the normal flags are not translated either. If the names for the straight flags need to be translatable, shouldn’t those for the normal ones be too?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These names are exposed, at least in the Symbols palette of the master palette. But indeed if the normal flags ain't, these don't need it either, so sorry for the noise.

@cbjeukendrup
Copy link
Contributor Author

Later today, I will implement that the Shadow Note follows the style of the score.

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

Successfully merging this pull request may close these issues.

3 participants