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 for Issue #16219: [MU4 Issue] Customise note input toolbar popup points to empty space #16226

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

octopols
Copy link
Contributor

@octopols octopols commented Feb 6, 2023

Resolves: #16219

Customise note input toolbar popup now points to the settings cog when the toolbar is set vertical.

  • I signed the CLA
  • The title of the PR describes the problem it addresses
  • Each commit's message describes its purpose and effects, and references the issue it resolves
  • If changes are extensive, there is a sequence of easily reviewable commits
  • The code in the PR follows the coding rules
  • There are no unnecessary changes
  • The code compiles and runs on my machine, preferably after each commit individually
  • I created a unit test or vtest to verify the changes I made (if applicable)

Changed `contentHeight`'s value to respond according to the input bar's orientation.
@octopols
Copy link
Contributor Author

octopols commented Feb 6, 2023

issue_16226_fix.mp4

@cbjeukendrup
Copy link
Contributor

cbjeukendrup commented Feb 6, 2023

This fix looks a bit naive to me. If you resize the MuseScore window to a smaller vertical size, then the bug will probably be back.

I think a real solution would be to move the arrow to the left side of the popup, in this case. That's a bit more complicated though.

@octopols
Copy link
Contributor Author

octopols commented Feb 6, 2023

I was just checking how the popup behaves with different screen sizes, having a fixed height even to 2 numbers like in this fix will cause some issues (look at the screenshot attached). I'll try to write a function which takes the application size as a parameter and changed the height accordingly.
When the toolbar is vertical I think you're right the best solution is to just put the arrow on the side of the popup.
Screenshot (42)
Screenshot (43)

@DmitryArefiev
Copy link
Contributor

@octopols @cbjeukendrup Hi! Is this ready for testing?

@octopols
Copy link
Contributor Author

@octopols @cbjeukendrup Hi! Is this ready for testing?

Sorry for the late reply but this fix does not completely solves the problem. I'm trying to work on the solution Casper suggested and I hope it doesn't take that long to implement.

@DmitryArefiev
Copy link
Contributor

Good. Thanks!

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.

[MU4 Issue] Customise note input toolbar popup points to empty space
3 participants