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 Gootville vtest #1776

Merged
merged 1 commit into from
Feb 17, 2015
Merged

fix Gootville vtest #1776

merged 1 commit into from
Feb 17, 2015

Conversation

Jojo-Schmitz
Copy link
Contributor

No description provided.

@lasconic
Copy link
Contributor

So we are storing Gonville for the musical font but Gootville Text for the text? that's weird.

@Jojo-Schmitz
Copy link
Contributor Author

No,Gootville.oft and GootvilleTxte.otf are just the filenames, used internally, in the UI they are called Gonville and Gonville Text.
Possible that this renaming a couple weeks ago had resulted in the setting of those test files to vanish.

@lasconic
Copy link
Contributor

The testfiles are currently containing reference to Gonville and Gonville Text.
The new testfiles would contain reference to Gonville and Gootville Text. Something fishy.

@Gootector
Copy link
Contributor

Your problem is with font names. You are using two different names: "Gonville" and "Gootville" and I don't know - why. My fonts are named "Gootville":
Gootville.otf
GootvilleText.otf
You must fix this this problem yourself.

@lasconic
Copy link
Contributor

thank you for your input...

@Gootector
Copy link
Contributor

:D Hold on!

@Jojo-Schmitz
Copy link
Contributor Author

this and the previous PR #1771 didn't change anything with regards to this. So this problem got to have been lurking since PR #1556, which did the renaming, (PR #1681 and #1577 just changed the font files). None of these 3 older ones ever updated the vtest files, they are still like they were before that renaming.

@lasconic
Copy link
Contributor

Yes, this is the problem. I think we don't get what we store in the file from the same place here.
https://github.com/musescore/MuseScore/blob/master/mscore/editstyle.cpp#L473

For score font, we store the first attribute from ScoreFont() here, which is also used in the UI. https://github.com/musescore/MuseScore/blob/master/libmscore/sym.cpp#L29 so Gonville for the Gootville font. The name of the font file is irrelevant, since it's the 3rd param in ScoreFont.

For musical text font, we store the second field from https://github.com/musescore/MuseScore/blob/master/mscore/editstyle.cpp#L740 so "Gootville Text" while the first one is used in the UI. The second field is also used as font family name. So here again the font filename is irrelevant but the font family in the file will make it work or not.

To sum it up, we are storing "Gootville Text" since https://github.com/musescore/MuseScore/pull/1556/files#diff-855615c9d487c6597383aed2fe12420cR740. I will annotate the code so we know all this for next time...

lasconic added a commit that referenced this pull request Feb 17, 2015
@lasconic lasconic merged commit 11b49de into musescore:master Feb 17, 2015
@Jojo-Schmitz Jojo-Schmitz deleted the gootville branch February 17, 2015 17:33
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.

None yet

3 participants