-
Notifications
You must be signed in to change notification settings - Fork 171
Add music view defaults in the Preferences dialog #1928
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
Add music view defaults in the Preferences dialog #1928
Conversation
This was requested in frescobaldi#1927.
|
I've tested it and it works as expected. Maybe I would change "Fit Both" to "Fit Page" in the Preferences, since the matching option in the dropdown menu is Page. |
This was requested in frescobaldi#1413.
This was also requested in frescobaldi#1413.
Done. I also added preferences for page layout and scrolling: It's not great UI design, but again my goal for tonight was just to get something in place and working. (Edit: I added some thoughts on what I think would be better under #1414.) |
|
How does this look? I also included the two settings that were moved from "Tools" in #1920, which I hope will make rebasing easier if we decide to merge this PR first. |
|
Looks good to me! Feel free to merge it when you think it's complete, then I can rebase #1920 |
|
@bmjcode I'm commenting in this PR even though you did not add the following line. Look at this Raster page layout option. What is this?!? Why is it in the section where you can set a single or double page layout? |
I put it in the preferences dialog because it's in the corresponding section of the Music menu. What it's for I can't tell you. It was there before I got here. |


This was requested in #1413. My thought was to get the functionality in place first, then tweak the design as part of #1414.
Edit: I originally cited #1927 but it looks like #1413 requested it first.