-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix typo in fonts_tablature.xml filename
- Loading branch information
Showing
4 changed files
with
3 additions
and
3 deletions.
There are no files selected for viewing
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4f5d638
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure this works: the file IS called fonts_tabulature.xml and the function StaffTypeTablature::readConfigFile() looks for such a name.
4f5d638
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this works well on my iOS device
4f5d638
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The file is not called fonts_tabulature.xml anymore. Tabulature is not a word in english. So I changed the name of the file to fonts_tablature.xml and I change the qrc files, the readConfigFile() accordingly. Tested on mac and windows.
I forgot to change mtest.qrc though. It's done in the next commit.
4f5d638
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I probably looked at the commit contents too quickly.
Anyway, for what is worth, "tabUlature" seems to be an English word: Wikipedia reports it as an alternative to "tablature", Goggle returns ca. 2,500,000 usages for it and the root it comes from gives several other words: tabUlar, (to) tabUlate, tabUlation, tabUlator...
4f5d638
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.