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 #289006: Expose "Select Instrument" in Inspector for Instrument Change text #5010

Merged

Conversation

IsaacWeiss
Copy link
Contributor

Addresses problem raised by Tantacrul in https://youtu.be/4hZxo96x48A?t=1149

@IsaacWeiss IsaacWeiss force-pushed the 289006-instrument-change-inspector branch from 15fa9e4 to c5af214 Compare May 10, 2019 22:37
@MarcSabatella
Copy link
Contributor

Nice! I should point out I also have a related complementary PR #4958, to allow staff name changes via Staff Properties. If/when that gets merged, we could consider adding a staff Properties button here too. Another possibility is just add instrument names fields to the Inspector, but that might require a little more hook up. @joshwd36 this is the kind of thing your project is about, right? Any ideas on what might makes sense as a short term solution?

@IsaacWeiss
Copy link
Contributor Author

Hmm, how about "Staff Properties..." and "Measure Properties..." buttons both being present whenever any note, rest, or range is selected (just like how they're in the menu when right-clicking on any note, rest, or empty space in a staff)?

When more than one staff or measure is in the range, the dialog could simply open to the first one.

@IsaacWeiss
Copy link
Contributor Author

That's for another PR, though.

@IsaacWeiss
Copy link
Contributor Author

Heyyyy, what if those buttons went in the status bar? 😲

@joshwd36
Copy link
Contributor

@MarcSabatella, yes, this is the kind of thing that my GSoC project is looking to fix. My idea was to have the instrument change dialog come up when the object is added, and also have a button in the inspector (and possibly the right click menu too). This would then change the staff properties automatically.

mscore/inspector/inspectorInstrchange.cpp Outdated Show resolved Hide resolved
mscore/inspector/inspector_instrchange.ui Show resolved Hide resolved
mscore/inspector/inspector_instrchange.ui Outdated Show resolved Hide resolved
mscore/propertymenu.cpp Outdated Show resolved Hide resolved
mscore/propertymenu.cpp Outdated Show resolved Hide resolved
// selectInstrument
//---------------------------------------------------------

void Ms::ScoreView::selectInstrument(InstrumentChange* ic)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The function name selectInstrument() is not very clear, if you don't know it's related to instrument change. Maybe something like selectNewInstrument() or simply changeInstrument() is better?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it should match the title of the dialog it opens, but that also could be "Change Instrument."

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, SelectInstrument is also the internal name of the object. I'm inclined to keep it that way.

@anatoly-os anatoly-os added this to the MuseScore 3.4 milestone Dec 20, 2019
@IsaacWeiss IsaacWeiss force-pushed the 289006-instrument-change-inspector branch from c5af214 to 7a32a58 Compare December 26, 2019 03:37
@IsaacWeiss
Copy link
Contributor Author

Updated and rebased

@Jojo-Schmitz
Copy link
Contributor

another rebase needed

@dmitrio95
Copy link
Contributor

This is partially included in #5782, and the inspector is being under a redesign now. Wouldn't it be better to wait for a redesign to happen in this case?

@dmitrio95 dmitrio95 added the strings Affects translatable strings label Mar 17, 2020
@IsaacWeiss IsaacWeiss force-pushed the 289006-instrument-change-inspector branch from 7a32a58 to 95e59f6 Compare March 17, 2020 16:58
mscore/propertymenu.cpp Outdated Show resolved Hide resolved
@IsaacWeiss IsaacWeiss force-pushed the 289006-instrument-change-inspector branch from 95e59f6 to 82f4334 Compare March 17, 2020 18:05
@dmitrio95 dmitrio95 merged commit 81767df into musescore:master Mar 18, 2020
@IsaacWeiss IsaacWeiss deleted the 289006-instrument-change-inspector branch April 26, 2020 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
strings Affects translatable strings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants