Selecting multiple elements with sameType but different subtype doesn't update inspector title
Reported version
3.1
Type
Graphical (UI)
Frequency
Once
Severity
S4 - Minor
Reproducibility
Always
Status
closed
Regression
Yes
Workaround
No
Project
Select both a slur and a tie and the inspector's title should be "Slur/Tie", this is the behaviour in 2.3.2, and the source .ui file has indeed "Slur/Tie". But now in 3.3.4 the title only says "Slur" or "Tie" based on the element first chosen.
Fix version
3.4.0
Comments
And it should change to "Slur" if only Slurs are selected and to "Tie" if only Ties are selected, but stay "Slur/Tie" if the's a mix of both being selected.
Likewise for Breaths and Caesuras.
The code for either is in place and worked in 2.x, but doesn't seem to any longer in 3.x
And now this should be done also for Articulations and Ornaments. See #298272: Inspector of ornaments is titled "Articulation"
In reply to And it should change to Slur… by Jojo-Schmitz
As soon as https://github.com/musescore/MuseScore/pull/5522 is merged ;-)
It is that way at least since 3.1, but probably ever since 3.0
See https://github.com/musescore/MuseScore/pull/5522.
Wait, the fix works but code logic needs to get sorted out...
By the way, is it really necessary to have a different inspector title for each breath or caesura? The title distinguishes breath from caesura, that's all it need to do! But right now with
accessibleInfo()
being the title, it in reality appears to be "Breath mark (comma)", "Breath mark (tick-like)" and so on. I think the information in the blanket is not really needed, maybe a judgement ofisBreath()
orisCaesura()
is enough, and the title being either "Breath" or "Caesura".And with this PR, there's another "problem" because of this: even if no more than two different types of breathes or caesuras are selected, the title becomes "Breath/Caesura" while one of the types isn't even selected.
Yes, Breath or Caesura would be good. I believe that's what it was in 2.x
In reply to Yes, Breath or Caesura would… by Jojo-Schmitz
Also done in the same PR.
Fixed in branch master, commit a5de5b78a8
_fix #298363: selecting multiple elements with sameType but different subtype doesn't update inspector title
Resolves: https://musescore.org/node/298363.
From 2.x to 3.x, a change was made which leaded to the issue: if
oSameTypes != sameTypes
, inspectors will not be updated. Even if slur and tie/breath and caesura/articulation and ornament are both selected, for each groupsameType
will continue to givetrue
, and so the inspector is not updated.This patch adds another field:
sameSubtypes
. IfsameTypes
istrue
but the subtypes are different, the inspector will be updated too._Fixed in branch master, commit 1279610c0b
_Merge pull request #5522 from Howard-C/ornament-inspector
[Inspector] fix #298272, fix #298363: inspector title issues_
Fixed in branch 3.4rc, commit fb18d06f9c
_Merge pull request #5522 from Howard-C/ornament-inspector
[Inspector] fix #298272, fix #298363: inspector title issues_
Automatically closed -- issue fixed for 2 weeks with no activity.