Bends: wrong font-style applied to new bends; font-style not carried over in 2.x files

• Jul 6, 2019 - 11:14
Reported version
3.2
Priority
P1 - High
Type
Functional
Frequency
Once
Severity
S3 - Major
Reproducibility
Always
Status
duplicate
Regression
Yes
Workaround
No
Project

OS: Windows 10 (10.0), Arch.: x86_64, MuseScore version (64-bit): 3.2.0.7613, revision: b64f852

  1. Open the attached 2.x file and take the reset (say);
  2. Compare the bends to the same in MS 2.3.2;
    Expected result: Should look identical between versions.
    Actual result: The text font is set to FreeSerif in MS 2, but has reset in 3.x to the first font in the list in the Inspector (on this system, Arial). Not only that, but if you open the Text Styles dialog you will say that the indicated font is indeed "FreeSerif."

Note also that the Bend Font "Reset to style default" button looks active but clicking it has no effect.

Related to #286108: [EPIC] Bends issues.

Attachment Size
prebend_ms_2.mscz 4.94 KB

Comments

Title Bends: displayed text is different from text indicated in text style dialog; and doesn't honor text style in imported 2.x files Bends: wrong font-style applied to new bends; text properties not carried over in 2.x files

If you correct the font-style of a new bend in the Inspector, then close and reopen the file, it defaults again to the first font-style in the list.

Title Bends: wrong font-style applied to new bends; text properties not carried over in 2.x files Bends: wrong font-style applied to new bends; font-style not carried over in 2.x files
Regression No Yes

The Bends tool is virtually unusable because the font-style cannot be changed from the default, and the default is the first font in alphabetical order.

What seems to be the case is that the style font setting is not actually hooked up to the element.

For me, though, while the reset button in the Inspector doesn't work (not surprising since we are missing the association to the style setting that would tell us the default), selecting a different font manually does work.

At first glance, it seems likely the problem is we are simply missing a line connecting Sid::bendFontFace to Pid::FONT_FACE in the definition of bendStyle at the top of bend.cpp.