Skip to content

Conversation

@SKefalidis
Copy link
Contributor

@SKefalidis SKefalidis commented Apr 7, 2020

Resolves: https://musescore.org/en/node/303422

This is a very simple fix to save the color of even-numbered lyrics.

The layout function behaves differently for odd and even lyrics. For odd-numbered lyrics the function returns almost immediately (using the first return statement). Even-numbered lyrics run through the entire function, one part of which sets the properties of the lyric to the defaults. I don't know if this is the desired behavior or a bug.

If it is a bug, that means that there are probably more problems than the colors not being imported, which might require some bigger changes. If that's the case I'll keep looking for a better solution to this problem.

edit: There are, indeed, more problems with importing (e.g. the font behaves weirdly). I will continue looking into this.

  • I signed CLA
  • I made sure the code in the PR follows the coding rules
  • I made sure the code compiles on my machine
  • I made sure there are no unnecessary changes in the code
  • I made sure the title of the PR reflects the core meaning of the issue you are solving
  • I made sure the commit message(s) contain a description and answer the question "Why do those changes fix that particular issue?" or "Why are those changes really necessary as improvements?"
  • I made sure the commit message title starts with "fix #424242:" if there is a related issue
  • [n\a] I created the test (mtest, vtest, script test) to verify the changes I made

@anatoly-os
Copy link
Contributor

@SKefalidis thank you for the research. Keep the changes made in current PR in mind when you are dealing with other "weirdnesses" of the even-numbered lines.

@lvinken might help to go deeper with musicXML import stuff.

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