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

MusicXML exporting: the wrong line-end type is exported for brackets. #18451

Closed
JosephBuchholz opened this issue Jul 6, 2023 · 0 comments · Fixed by #19455
Closed

MusicXML exporting: the wrong line-end type is exported for brackets. #18451

JosephBuchholz opened this issue Jul 6, 2023 · 0 comments · Fixed by #19455
Assignees
Labels
MusicXML needs review The issue needs review to set priority, fix version or change status etc.

Comments

@JosephBuchholz
Copy link

Issue type

Other type of issue

Bug description

Brackets with a line-end type of 'both' are not exported to MusicXML correctly. Instead the line-end type is exported as 'down':

<direction placement="below">
  <direction-type>
    <bracket type="stop" number="1" line-end="down" end-length="6"/>
  </direction-type>
</direction>

should be:

<direction placement="below">
  <direction-type>
    <bracket type="stop" number="1" line-end="both" end-length="6"/>
  </direction-type>
</direction>

Steps to reproduce

  1. Create a MuseScore project that has a bracket with a double line end. For example a let-ring or P.M. element
  2. Export to MusicXML
  3. Import the exported MusicXML file back in to MuseScore
  4. Notice the line end for the bracket has changed

Screenshots/Screen recordings

Before export:
Screenshot 2023-07-06 133249
After export:
Screenshot 2023-07-06 133224

MuseScore Version

OS: Windows 10 Version 2009, Arch.: x86_64, MuseScore version (64-bit): 4.0.2-230651553, revision: github-musescore-musescore-dbe7c6d

Regression

I don't know

Operating system

Windows 11

Additional context

The MuseScore and MusicXML files:
files.zip

@muse-bot muse-bot added the needs review The issue needs review to set priority, fix version or change status etc. label Jul 6, 2023
cbjeukendrup added a commit that referenced this issue Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MusicXML needs review The issue needs review to set priority, fix version or change status etc.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

5 participants