Fix updating the Offset property in the Inspector when dragging item in the notation

Author: cbjeukendrup

src/inspector/models/abstractinspectorproxymodel.h

@@ -63,6 +63,8 @@ class AbstractInspectorProxyModel : public AbstractInspectorModel
 
     void updateModels(const ElementKeySet& newElementKeySet);
 
+    void onCurrentNotationChanged() override;
+
 public slots:
     void setDefaultSubModelType(mu::inspector::InspectorModelType modelType);
 
@@ -74,8 +76,6 @@ public slots:
     void setModels(const QList<AbstractInspectorModel*>& models);
 
 private:
-    void onCurrentNotationChanged() override;
-
     QHash<InspectorModelType, AbstractInspectorModel*> m_modelsHash;
     InspectorModelType m_defaultSubModelType = InspectorModelType::TYPE_UNDEFINED;
 };

src/inspector/models/general/generalsettingsmodel.cpp

@@ -33,8 +33,12 @@ GeneralSettingsModel::GeneralSettingsModel(QObject* parent, IElementRepositorySe
 
     setTitle(qtrc("inspector", "General"));
     setSectionType(InspectorSectionType::SECTION_GENERAL);
-    setPlaybackProxyModel(new PlaybackProxyModel(this, repository));
-    setAppearanceSettingsModel(new AppearanceSettingsModel(this, repository));
+
+    m_playbackProxyModel = new PlaybackProxyModel(this, repository);
+    m_playbackProxyModel->init();
+
+    m_appearanceSettingsModel = new AppearanceSettingsModel(this, repository);
+    m_appearanceSettingsModel->init();
 }
 
 void GeneralSettingsModel::createProperties()
@@ -124,6 +128,14 @@ void GeneralSettingsModel::loadProperties(const mu::engraving::PropertyIdSet& pr
     }
 }
 
+void GeneralSettingsModel::onCurrentNotationChanged()
+{
+    AbstractInspectorModel::onCurrentNotationChanged();
+
+    m_appearanceSettingsModel->onCurrentNotationChanged();
+    m_playbackProxyModel->onCurrentNotationChanged();
+}
+
 void GeneralSettingsModel::onVisibleChanged(bool visible)
 {
     beginCommand();
@@ -167,15 +179,3 @@ QObject* GeneralSettingsModel::appearanceSettingsModel() const
 {
     return m_appearanceSettingsModel;
 }
-
-void GeneralSettingsModel::setPlaybackProxyModel(PlaybackProxyModel* playbackProxyModel)
-{
-    m_playbackProxyModel = playbackProxyModel;
-    emit playbackProxyModelChanged(m_playbackProxyModel);
-}
-
-void GeneralSettingsModel::setAppearanceSettingsModel(AppearanceSettingsModel* appearanceSettingsModel)
-{
-    m_appearanceSettingsModel = appearanceSettingsModel;
-    emit appearanceSettingsModelChanged(m_appearanceSettingsModel);
-}

src/inspector/models/general/generalsettingsmodel.h

@@ -36,8 +36,8 @@ class GeneralSettingsModel : public AbstractInspectorModel
     Q_PROPERTY(PropertyItem * isPlayable READ isPlayable CONSTANT)
     Q_PROPERTY(PropertyItem * isSmall READ isSmall CONSTANT)
 
-    Q_PROPERTY(QObject * playbackProxyModel READ playbackProxyModel NOTIFY playbackProxyModelChanged)
-    Q_PROPERTY(QObject * appearanceSettingsModel READ appearanceSettingsModel NOTIFY appearanceSettingsModelChanged)
+    Q_PROPERTY(QObject * playbackProxyModel READ playbackProxyModel CONSTANT)
+    Q_PROPERTY(QObject * appearanceSettingsModel READ appearanceSettingsModel CONSTANT)
 
 public:
     explicit GeneralSettingsModel(QObject* parent, IElementRepositoryService* repository);
@@ -50,13 +50,7 @@ class GeneralSettingsModel : public AbstractInspectorModel
     QObject* playbackProxyModel() const;
     QObject* appearanceSettingsModel() const;
 
-public slots:
-    void setPlaybackProxyModel(mu::inspector::PlaybackProxyModel* playbackProxyModel);
-    void setAppearanceSettingsModel(mu::inspector::AppearanceSettingsModel* appearanceSettingsModel);
-
-signals:
-    void playbackProxyModelChanged(QObject* playbackProxyModel);
-    void appearanceSettingsModelChanged(QObject* appearanceSettingsModel);
+    void onCurrentNotationChanged() override;
 
 private:
     void createProperties() override;