Merge pull request #4954 from mattmcclinch/116771-toggle-visible

fix #116771: Note's dot's visibility is incorrect when toggling visibility with note and dot selected
musescore · May 1, 2019 · 42bce14 · 42bce14
2 parents 0b3cfbc + 09057c0
commit 42bce14
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/libmscore/cmd.cpp b/libmscore/cmd.cpp
@@ -3519,6 +3519,9 @@ void Score::cmdToggleVisible()
       for (Element* e : selection().elements()) {
             if (e->isBracket())     // ignore
                   continue;
+            if (e->isNoteDot() && selection().elements().contains(e->parent()))
+                  // already handled in Note::setProperty() and Rest::setProperty(); don't toggle twice
+                  continue;
             bool spannerSegment = e->isSpannerSegment();
             if (!spannerSegment || !spanners.contains(toSpannerSegment(e)->spanner()))
                   e->undoChangeProperty(Pid::VISIBLE, !e->getProperty(Pid::VISIBLE).toBool());