Percussion panel - assorted small fixes:

- Fixed notation preview background colour
- Fixed alignment of disabled state text
- Fixed panel not updating properly after selecting an unpitched stave
- Ensured keyboard focus moves to pads when entering/exiting edit layout mode
- Fixed clipped navigation outline for add row button
- Fixed sounds not previewing properly
- Added missing overrides

Author: mathesoncalum

src/notation/internal/notationconfiguration.h

@@ -207,11 +207,11 @@ class NotationConfiguration : public INotationConfiguration, public muse::async:
     muse::async::Notification autoShowPercussionPanelChanged() const override;
 
     bool showPercussionPanelPadSwapDialog() const override;
-    void setShowPercussionPanelPadSwapDialog(bool show);
+    void setShowPercussionPanelPadSwapDialog(bool show) override;
     muse::async::Notification showPercussionPanelPadSwapDialogChanged() const override;
 
     bool percussionPanelMoveMidiNotesAndShortcuts() const override;
-    void setPercussionPanelMoveMidiNotesAndShortcuts(bool move);
+    void setPercussionPanelMoveMidiNotesAndShortcuts(bool move) override;
     muse::async::Notification percussionPanelMoveMidiNotesAndShortcutsChanged() const override;
 
     muse::io::path_t styleFileImportPath() const override;

src/notation/qml/MuseScore/NotationScene/PercussionPanel.qml

@@ -415,6 +415,7 @@ Item {
                 orientation: Qt.Horizontal
 
                 navigation.panel: addRowButtonPanel
+                drawFocusBorderInsideRect: true
 
                 onClicked: {
                     padGrid.model.addEmptyRow()
@@ -434,6 +435,7 @@ Item {
 
             font: ui.theme.bodyFont
             text: qsTrc("notation/percussion", "Select an unpitched percussion staff to see available sounds")
+            verticalAlignment: Text.AlignVCenter
         }
     }
 

src/notation/view/paintedengravingitem.cpp

@@ -88,7 +88,7 @@ void PaintedEngravingItem::paintNotationPreview(muse::draw::Painter& painter, qr
 
     params.drawStaff = true;
 
-    painter.fillRect(params.rect, configuration()->scoreInversionColor());
+    painter.fillRect(params.rect, configuration()->noteBackgroundColor());
 
     EngravingItemPreviewPainter::paintPreview(m_item.get(), params);
 }

src/notation/view/percussionpanel/percussionpanelmodel.cpp

@@ -21,6 +21,9 @@
  */
 
 #include "percussionpanelmodel.h"
+
+#include "notation/utilities/percussionutilities.h"
+
 #include "types/translatablestring.h"
 #include "ui/view/iconcodes.h"
 
@@ -169,6 +172,7 @@ void PercussionPanelModel::handleMenuItem(const QString& itemId)
     } else if (itemId == EDIT_LAYOUT_CODE) {
         const bool currentlyEditing = m_currentPanelMode == PanelMode::Mode::EDIT_LAYOUT;
         currentlyEditing ? finishEditing() : setCurrentPanelMode(PanelMode::Mode::EDIT_LAYOUT);
+        currentlyEditing ? m_padListModel->focusFirstActivePad() : m_padListModel->padFocusRequested(0);
     } else if (itemId == RESET_LAYOUT_CODE) {
         resetLayout();
     }
@@ -190,13 +194,13 @@ void PercussionPanelModel::finishEditing(bool discardChanges)
     Instrument* inst = instAndPart.first;
     Part* part = instAndPart.second;
 
-    IF_ASSERT_FAILED(inst && inst->drumset() && part) {
+    if (discardChanges) {
+        m_padListModel->setDrumset(inst ? inst->drumset() : nullptr);
+        setCurrentPanelMode(m_panelModeToRestore);
         return;
     }
 
-    if (discardChanges) {
-        m_padListModel->setDrumset(inst->drumset());
-        setCurrentPanelMode(m_panelModeToRestore);
+    IF_ASSERT_FAILED(inst && inst->drumset() && part) {
         return;
     }
 
@@ -435,26 +439,17 @@ void PercussionPanelModel::writePitch(int pitch)
 
 void PercussionPanelModel::playPitch(int pitch)
 {
-    const mu::engraving::InputState& inputState = score()->inputState();
-    if (!inputState.cr()) {
+    if (!interaction()) {
         return;
     }
 
-    Chord* chord = mu::engraving::Factory::createChord(inputState.lastSegment());
-    chord->setParent(inputState.lastSegment());
-
-    Note* note = mu::engraving::Factory::createNote(chord);
-    note->setParent(chord);
-
-    note->setStaffIdx(mu::engraving::track2staff(inputState.cr()->track()));
-
-    const mu::engraving::NoteVal nval = score()->noteVal(pitch, /*transpose*/ false);
-    note->setNval(nval);
+    const NoteInputState inputState = interaction()->noteInput()->state();
+    std::shared_ptr<Chord> chord = PercussionUtilities::getDrumNoteForPreview(m_padListModel->drumset(), pitch);
 
-    playbackController()->playElements({ note });
+    chord->setParent(inputState.segment);
+    chord->setTrack(inputState.currentTrack);
 
-    note->setParent(nullptr);
-    delete note;
+    playbackController()->playElements({ chord.get() });
 }
 
 void PercussionPanelModel::resetLayout()

src/notation/view/percussionpanel/percussionpanelpadlistmodel.cpp

@@ -225,6 +225,16 @@ mu::engraving::Drumset PercussionPanelPadListModel::constructDefaultLayout(const
     return defaultLayout;
 }
 
+void PercussionPanelPadListModel::focusFirstActivePad()
+{
+    for (int i = 0; i < m_padModels.size(); i++) {
+        if (m_padModels.at(i)) {
+            emit padFocusRequested(i);
+            return;
+        }
+    }
+}
+
 void PercussionPanelPadListModel::focusLastActivePad()
 {
     for (int i = m_padModels.size() - 1; i >= 0; --i) {

src/notation/view/percussionpanel/percussionpanelpadlistmodel.h

@@ -81,6 +81,7 @@ class PercussionPanelPadListModel : public QAbstractListModel, public muse::Inje
     int nextAvailableIndex(int pitch) const; //! NOTE: This may be equal to m_padModels.size()
     int nextAvailablePitch(int pitch) const;
 
+    void focusFirstActivePad();
     void focusLastActivePad();
 
     muse::async::Notification hasActivePadsChanged() const { return m_hasActivePadsChanged; }