MuseScore icon indicating copy to clipboard operation
MuseScore copied to clipboard

[MU4 Task] Add 'Beam type' to Properties panel for rests

Open its-not-nice opened this issue 2 years ago • 5 comments

Task description The properties panel has a 'Beam type' section for Notes: image

This should be added for Rests too. At the moment, only the 'Beam properties' palette can be used to apply these to rests. For notes, one can consult the properties panel to see which 'beam type' has been applied to a note, but one cannot do this for rests.

(I can't shake the feeling that there's already an issue/task for this but I can't seem to find it. If anyone does, close this one and shout at me.)

its-not-nice avatar Dec 20 '22 16:12 its-not-nice

In MU4 the rests don't have any specific section for them in Properties, whereas in MU3 they have a section that only contains the "Small" option, but I guess that's covered more consistently in MU4. So this here seems to need the design for a new "Rest" section for Properties? Can't think of anything else though that might need to go in that section besides reusing the "Beam type" panel there.

HowardC2019 avatar Dec 20 '22 17:12 HowardC2019

image

I played with this for a while and saw that if I select both the first note and the first rest here, setting the beam mode in Properties would make the note's beam mode change, but not the rest's, so I went into the code and added one line that made the rest's beam mode able to change, but still if I only select the rest, there isn't any setting in Properties.

It can definitely be fixed by adding a "Rest" section, in which the "Beam type" panel is reused, but then there's the question of whether or not the "Rest" section is shown when there're both note(s) and rest(s) selected, since the one single setting available in "Rest" is already covered in the "Note" section. It's one option to just show the "Note" section, and only show "Rest" if no notes are selected, but if in some cases it might be desired to set the beam mode of notes and rests separately (is it?), the two sections should still both appear.

Anyway, I may seem overzealous about this, if so, I apologise, but I think either way, I would be able to work on this issue.

HowardC2019 avatar Dec 20 '22 17:12 HowardC2019

there's the question of whether or not the "Rest" section is shown when there're both note(s) and rest(s) selected

We already have a design solution for this, which is to show buttons for each property type when more than one different element is selected.

Screenshot 2023-01-06 at 2 43 22 pm

So I think we fall back on this in adding a new properties section for "rests".

The fact that you can only open one of these popups at a time should hopefully allay any confusion concerning the adjustment of beaming for either notes or rests (because you can only do either one of these at a time).

Personally, I think a dedicated "rests" section makes sense here, because it seems intuitive to want to modify a beam over a rest by first selecting the rest.

@oktophonie can you think of any other properties we might need for rests? I might give this one a go now.

bkunda avatar Jan 06 '23 13:01 bkunda

(I can't shake the feeling that there's already an issue/task for this but I can't seem to find it. If anyone does, close this one and shout at me.)

@oktophonie was this the task you were thinking of? https://github.com/musescore/MuseScore/issues/9892 I was certainly reminded of it. This was to add an option for "beam end".

bkunda avatar Jan 06 '23 13:01 bkunda

@bkunda For now, not that immediately springs to mind; but when we get to implementing stemlets it would make sense for control(s) for those to live under Rest -> Beam too.

its-not-nice avatar Jan 06 '23 14:01 its-not-nice

Closing as subsumed by https://github.com/musescore/MuseScore/issues/15779

its-not-nice avatar Feb 23 '23 12:02 its-not-nice