Settings in the inspector that are supposed to be greyed out don't appear as such upon the next click on the element

• Nov 1, 2019 - 14:43
Reported version
3.2
Type
Graphical (UI)
Frequency
Once
Severity
S4 - Minor
Reproducibility
Always
Status
closed
Regression
No
Workaround
No
Project

For example, for fermatas, untick "Play" greys out time stretch, but the next time you click the fermata and look in the inspector, time stretch is enabled again.


Comments

Not sure if it's related, but really, pretty much no changes made other than via the specific control you are are using will reflect in the Inspector. There's a line at the top of Inspector::update() that returns immediately if _inspectorEdit is true, thus preventing changes initiated by other inspector control from updating the view. This would be the place to catch them globally, but I suspect bad things would happen if this line were removed.

As for catching these cases one at a time, the one I know of I'd love to see fixed is that changing offset for autopalced elements can potentially affect min distance, and it would be nice if that updated directly.

In reply to by Marc Sabatella

There is already a fix suggested by @mattmcclinch, see here. The real problem is that the connection is successful but it isn't remembered, so I don't know whether that line in update() is related, it's not convenient for me to check the source code now.

Offset affecting min distance sounds like a totally different issue though.

Well, it's similar in that there is a discrepancy betwene what happens while editing in the Inspector and what happens if you click off the element and back on, because the Inspector doesn't fully update while editing in the Inspector. The is potentially true whether we are talking about disabling settings or changing them.

Status PR created fixed

Fixed in branch master, commit c46523d668

_fix #296415: settings in the inspector that are supposed to be greyed out don't appear as such upon the next click on the element

Resolves: https://musescore.org/node/296415.

The inspector is not updated properly upon the next click, which can be solved by adding a few lines in setElement() stating that those which are supposed to be greyed out (or invisible) have their enabled (or visible) set to false.

Co-Authored-By: mattmcclinch _

Fixed in branch master, commit 556a744edb

_Merge pull request #5447 from Howard-C/inspector-grey-out

fix #296415: settings in the inspector that are supposed to be greyed out don't appear as such upon the next click on the element_

Fix version
3.3.3