Skip to content

Conversation

@bmjcode
Copy link
Collaborator

@bmjcode bmjcode commented Apr 19, 2025

This was requested in #1413. My thought was to get the functionality in place first, then tweak the design as part of #1414.

image

Edit: I originally cited #1927 but it looks like #1413 requested it first.

@fedelibre
Copy link
Member

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.

@bmjcode
Copy link
Collaborator Author

bmjcode commented Apr 23, 2025

Maybe I would change "Fit Both" to "Fit Page" in the Preferences, since the matching option in the dropdown menu is Page.

Done.

I also added preferences for page layout and scrolling:

image

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.)

@bmjcode
Copy link
Collaborator Author

bmjcode commented Apr 24, 2025

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.

image

@bmjcode bmjcode changed the title Add a preference for default display scaling Add music view defaults in the Preferences dialog Apr 24, 2025
@fedelibre
Copy link
Member

Looks good to me!

Feel free to merge it when you think it's complete, then I can rebase #1920

@bmjcode bmjcode merged commit b94c0eb into frescobaldi:master Apr 26, 2025
@bmjcode bmjcode deleted the musicview-initial-scale-preferences branch April 26, 2025 12:43
@fedelibre
Copy link
Member

@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?

@bmjcode
Copy link
Collaborator Author

bmjcode commented May 4, 2025

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.

@fedelibre
Copy link
Member

fedelibre commented May 4, 2025

There's a comment in musicviewers.py saying:

These are from qpageview.view.ViewProperties

In fact the code is here.

And the raster layout is explained here.
I think that preference deserves a tooltip and an explanation in the user manual. I'll test it and open a new PR.

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.

2 participants