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

Fix GH#25048: Export Tempo as system text in MusicXML #689

Open
wants to merge 1 commit into
base: 3.x
Choose a base branch
from

Conversation

Jojo-Schmitz
Copy link
Owner

@Jojo-Schmitz Jojo-Schmitz commented Nov 8, 2024

Backport of musescore#25312

Resolves: musescore#25048 and and musescore#17431

@Jojo-Schmitz Jojo-Schmitz force-pushed the 3.x-xml/systemtempo branch 3 times, most recently from 6f48ffa to 4cdb621 Compare November 8, 2024 20:34
_xml.stag(QString("direction placement=\"%1\"").arg((text->placement() == Placement::BELOW ) ? "below" : "above"));
QString tempoAttrs = QString("direction placement=\"%1\"").arg(text->placement() == Placement::BELOW ? "below" : "above");
if (text->systemFlag())
tempoAttrs += QString(" system=\"%1\"").arg(text->isLinked() ? "also-top" : "only-top");
Copy link
Owner Author

Choose a reason for hiding this comment

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

This "also-top" seems to never happen with Mu3 ?!?

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.

2 participants