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

"Parts" not accessible from any menu #23479

Open
skdonnelly opened this issue Jul 4, 2024 · 12 comments · May be fixed by #24039
Open

"Parts" not accessible from any menu #23479

skdonnelly opened this issue Jul 4, 2024 · 12 comments · May be fixed by #24039
Labels
community Issues particularly suitable for community contributors to work on good first issue Issues suitable for first-time contributors. See https://github.com/musescore/MuseScore/contribute P3 Priority: Low UI Visual issues affecting the UI (not notation)

Comments

@skdonnelly
Copy link

Issue type

UI bug (incorrect info or interface appearance)

Bug description

No response

Steps to reproduce

Documentation refers to "Parts" being in the "File" menu, but it is not there. It should be in a menu (like everything) but it should be in the View menu for consistency with the Mixer button which is right beside it in the toolbar and appears in the View menu.

Screenshots/Screen recordings

image
image

MuseScore Version

MuseScore Studio version (64-bit): 4.3.2-241630831, revision: github-musescore-musescore-22b46f2

Regression

I don't know

Operating system

OS: Windows 10 Version 2009 or later, Arch.: x86_64,

Additional context

No response

@muse-bot muse-bot added the UI Visual issues affecting the UI (not notation) label Jul 4, 2024
@MarcSabatella
Copy link
Contributor

The documentation shouldn't say that, because indeed, by design, it is no longer there. Are you sure you are looking at the current version of the Handbook and not an older one? Can you post a link to where you are seeing that outdated info?

@cbjeukendrup
Copy link
Contributor

I think it would make sense to add it back to the file menu, just below project properties.

@cbjeukendrup cbjeukendrup added good first issue Issues suitable for first-time contributors. See https://github.com/musescore/MuseScore/contribute community Issues particularly suitable for community contributors to work on labels Jul 5, 2024
@bkunda bkunda moved this to First release after the upcoming one in MuseScore Studio Backlog Jul 5, 2024
@bkunda bkunda added the P3 Priority: Low label Jul 5, 2024
@bkunda bkunda unassigned Eism Jul 5, 2024
@skdonnelly
Copy link
Author

The reference was in the Musescore 3 documentation I realize now, the Musescore 4 documentation is correct.

I still think it is more consistent to have it in the View menu, since the Mixer button corresponds with View > Mixer, the Parts button right beside it should correspond to View > Parts for a consistent user experience. Users shouldn't have to surf through all the menus to find the way to view a specific dialog.

@cbjeukendrup
Copy link
Contributor

I'm not a fan of putting it in the View menu, because it has nothing to do with "View". The mixer has a clear reason for being in the View menu, because that menu is about all the side panels and bottom panels that can be opened and closed. The Parts dialog is not such a panel, and fits better together with "Project properties". This should actually reduce the amount of "surfing through all the menus", because it means that options are where they logically belong.

Also, using that top-center toolbar as a guideline for which things need to be located together in the menus makes no sense for two reasons:

  • if someone can just use that toolbar, they would do that instead of looking in any menu, so even if the toolbar causes expectations about which things could be in the same menu, those expectations will never have any effect
  • currently, that toolbar contains indeed only these two random buttons, but in the future it will include at least a third and probably also a fourth button, and then it will all make more sense

@MarcSabatella
Copy link
Contributor

Several other things in the View menu are not side/bottom panels either - Master palette, Instruments, Playback setup. I'm not saying that's good, though - I agree it's kind of awkward since none of those are about what a View menu is more typically about. But, none of those are really great candidates for "File" to me, either.

Maybe all of these could be moved to a new menu? Certainly "Instruments" and "Parts" seem like they belong together at least as much as "Project properties" and "Parts" do. Master palette maybe less so. I don't know, but it definitely feels like design input would be useful here.

@skdonnelly
Copy link
Author

I think the Parts dialog could/should just be another palette and be in the View menu with the rest. It's no different from the Instrument properties palette really.

@loss-of-generality
Copy link

loss-of-generality commented Jul 31, 2024

I understand that there is some design input needed for this issue but I would be interested in working on this. Should I wait until there is general consensus on the design or should I go ahead and start working?

@pacebes
Copy link
Contributor

pacebes commented Jul 31, 2024

Hi

I am looking for a "good first issue".
I have made some changes to a MuseScore in order to add this "Parts" section just below Project properties". See:

PartsInaMenu

As far as I understand this change is yet under discussion (if it should be made and where should the option be included) . Am I right ??

Thanks

@cbjeukendrup
Copy link
Contributor

@pacebes That's right. If it will be put into the File menu, I would suggest removing the separator line between project properties and parts.

Also, the title of the menu item should be Parts…, with an ellipsis character added to it, just like Project properties. That's because the command opens a dialog, and the convention is to add an ellipsis character to the name such commands.
However, if you add the ellipsis in notationuiactions.cpp, then it will also affect the text of the button in the main window, which is not desirable.
Fortunately, you can solve it by editing only appmenumodel.cpp. The makeMenuItem method that is inherited from AbstractMenuModel namely has a second parameter, which allows specifying the title in the form of a TranslatableString. For example, TranslatableString("action", "Parts…").

After implementing these two suggestions, feel free to open a PR with your changes, even though there is no consensus yet about in which menu the item should appear. After you open a PR, you can still easily modify it, it the designers decide to put the item in a different menu. Thanks in advance!

@pacebes
Copy link
Contributor

pacebes commented Aug 14, 2024

Thanks @cbjeukendrup
I will open that PR.

pacebes added a commit to pacebes/MuseScore that referenced this issue Aug 14, 2024
@pacebes pacebes linked a pull request Aug 14, 2024 that will close this issue
8 tasks
@pacebes
Copy link
Contributor

pacebes commented Aug 14, 2024

@cbjeukendrup , I have created the pull request: #24039

I don't know if I should set the status to "PR created" or where may I do that:

_ https://musescore.org/es/handbook/developers-handbook/finding-your-way-around/git-workflow#Write_some_code! :
If you are fixing an issue from the tracker, set the issue's status to "PR created" with a link to your pull request on GitHub. One of the developers with push rights on the main repo will merge your request ASAP, or leave a comment to say why it hasn't been merged and what needed to be done to get it merged.

Kind regards

@cbjeukendrup
Copy link
Contributor

Ah, that's about the old issue tracker that we used to have on MuseScore.org. So no action is needed.

At some point we should move all remaining useful content of that "developer handbook" to the Wiki here on GitHub, and then archive those pages on MuseScore.org. That's going to be a big task...

@cbjeukendrup cbjeukendrup moved this from Available to In progress in Community Projects Aug 14, 2024
@cbjeukendrup cbjeukendrup moved this from First release after the upcoming one to Already in progress in MuseScore Studio Backlog Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Issues particularly suitable for community contributors to work on good first issue Issues suitable for first-time contributors. See https://github.com/musescore/MuseScore/contribute P3 Priority: Low UI Visual issues affecting the UI (not notation)
Projects
Status: In progress
Status: Already in progress
Development

Successfully merging a pull request may close this issue.

8 participants