MS3: Line color not being saved.
Reported version
3.0
Priority
P0 - Critical
Type
Graphical (UI)
Frequency
Once
Severity
S3 - Major
Reproducibility
Always
Status
closed
Regression
Yes
Workaround
No
Project
OS: macOS 10.14, Arch.: x86_64, MuseScore version (64-bit): 3.0.1.20439, revision: 06a66a2
Make a "note anchored line" (or any line as far as I know). Set its "line color" (and its "element color" -- what is this?) to Maraschino/red. Save the file. Close it. Reopen it. The lines are black/Grey.
Attachment | Size |
---|---|
line_test.mscz | 5.31 KB |
Fix version
3.0.3
Comments
Confirmed. Could be a mismatch between the properties for the line and for the line "segment" (since lines can continue across systems, each segment has its own properties for things like offset and color). Which also suggests that at the same time, we should probably consider making the element color automatically propagate to the correct line / segment properties so it doesn't just sit there doing nothing.
I suggest that we get rid of the 'line colour' property altogether, and just use the normal Pid::COLOR (which is what 'element colour' is linked to) to set the _lineColor of a Line. I'd keep the 'Line Color' field in the inspector textlinebase though, just instead of linking it to Pid::LINE_COLOR, it'd be linked to Pid::COLOR.
In reply to I suggest that we get rid of… by TheOtherJThistle
Did you delete your comment? I'll respond anyway: the top colour field is one that exists for every element in the score, text, notes, barlines, etc. The bottom colour field is one specific to lines. Looking through the code, I don't see any reason why we can't get rid of the bottom colour field, and just link up the line's colour attribute to the top colour field instead. So, that's what I'm doing. I'll submit a PR in ten minutes or so.
In reply to Did you delete your comment?… by TheOtherJThistle
Yes, thanks. When I got back here I saw that Marc had raised this issue already. It's high time for a new download.
https://github.com/musescore/MuseScore/pull/4697
Fixed in branch master, commit 43b1d5f050
fix #282404: line colour not saved
Fixed in branch master, commit efdf43d96f
_Merge pull request #4697 from jthistle/282404-line-color-save
fix #282404: line colour not saved_
Fixed in branch 3.0.x, commit ef56ef8bc6
_Merge pull request #4697 from jthistle/282404-line-color-save
fix #282404: line colour not saved_
Automatically closed -- issue fixed for 2 weeks with no activity.