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

Status Bar - Zoom - Entering values is limited to 25% #16701

Closed
DmitryArefiev opened this issue Mar 8, 2023 · 2 comments · Fixed by #16713
Closed

Status Bar - Zoom - Entering values is limited to 25% #16701

DmitryArefiev opened this issue Mar 8, 2023 · 2 comments · Fixed by #16713
Assignees
Labels
P3 Priority: Low regression MS4 Regression on a prior release UI Visual issues affecting the UI (not notation)

Comments

@DmitryArefiev
Copy link
Contributor

DmitryArefiev commented Mar 8, 2023

Steps to reproduce

  1. Go to Status Bar
  2. Double click on zoom % in bottom right
  3. Enter 10 or 5 and hit Return - it changes to 25%

Expected result: It should be limited to 5% (the same limit value as Zoom minus control has)

Screenshots/Screen recordings

bandicam.2023-03-08.13-07-55-589.mp4

MuseScore Version

4.0.2

Regression

Yes, this used to work in Musescore 3.x and now is broken

Operating system

Win/Mac/Lin

@Tantacrul Tantacrul added regression MS4 Regression on a prior release UI Visual issues affecting the UI (not notation) labels Mar 8, 2023
@DmitryArefiev DmitryArefiev added the P3 Priority: Low label Mar 8, 2023
@limse10
Copy link
Contributor

limse10 commented Mar 8, 2023

Do we want to change the min zoom percentage to 5%? Or was there a reason it is set at 25%? If we do want to change it, I can work on that!

@Eism Eism assigned limse10 and unassigned Eism Mar 8, 2023
limse10 added a commit to limse10/MuseScore that referenced this issue Mar 8, 2023
limse10 added a commit to limse10/MuseScore that referenced this issue Mar 30, 2023
DmitryArefiev added a commit that referenced this issue Mar 31, 2023
Fix #16701: set MIN_DISPLAYED_ZOOM_PERCENTAGE to 5
@DmitryArefiev
Copy link
Contributor Author

Fixed in #16713

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P3 Priority: Low regression MS4 Regression on a prior release UI Visual issues affecting the UI (not notation)
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants