Allow for asynchronous inspector updates

[bluebear94]

Resolves: #14383

Previously, updating the inspector model would cause QML components to be created synchronously, causing a delay when selecting certain elements. We set an incubation controller on the QQmlEngine to allow for objects to be created asynchronously when requested. To avoid blocking the main thread when updating the inspector, we also replace the StyledFlickable + Repeater combo with a StyledListView.

Current bugs: the space above the separator lines is missing

• I signed the CLA
• The title of the PR describes the problem it addresses
• Each commit's message describes its purpose and effects, and references the issue it resolves
• If changes are extensive, there is a sequence of easily reviewable commits
• The code in the PR follows the coding rules
• There are no unnecessary changes
• The code compiles and runs on my machine, preferably after each commit individually
• I created a unit test or vtest to verify the changes I made (if applicable) – N/A

[cbjeukendrup]

This looks good to me now! Are you planning to make further changes in this PR, or is it time to mark it 'ready for review' so that I can approve it?

[bluebear94]

I don’t have any further changes planned.

[RomanPudashkin]

@bluebear94 please rebase your PR to fix the red vtests, thanks!

[DmitryArefiev]

Tested on Win10, Mac13.6, LinuxUbuntu24.04.1 LTS

@bluebear94 Sorry for delay in testing. I found a regression on Mac with Inspector updating when using select all and switching between panels

1. Make sure MS opens with Palette panel by default after opening score
2. Run MS
3. Open
   inspector .zip (Palettes panel is opened)
4. Cmd+A (select all)
5. Switch to Properties panel
6. Scroll down
7. Click on empty space in score - Properties panel becomes totally empty

Screen.Recording.2024-10-23.at.16.38.51.2.mp4

[cbjeukendrup]

Perhaps need to call returnToBounds() at some point, maybe onContentHeightChanged.
Or add something like cacheBuffer: 3 * height (see https://doc.qt.io/qt-6/qml-qtquick-listview.html#cacheBuffer-prop).

[bluebear94]

@DmitryArefiev I don’t have macOS, so let me know if one of @cbjeukendrup’s suggestions fixes the issue for you.

[DmitryArefiev]

@bluebear94 I'm not a developer (can't change the code on my side).. You can push those suggestions sequentially and I'll check it on macOS

[bluebear94]

I’ve pushed the first suggestion; let me know whether it fixes the bug on macOS.

[cbjeukendrup]

@bluebear94 I tried it out and this seems to work for me. However, I noticed some QML warnings, and some more code that turns out to become redundant given your changes, so I created a few commits here: master...cbjeukendrup:MuseScore:properties/optimisations
You could try out that branch, to be sure that it doesn't regress the performance improvements from your changes, and if it's okay, you can cherry-pick those commits to your branch.

Additionally, it would be good to rebase your PR branch: git pull --rebase upstream master && git push -f

[bluebear94]

Unfortunately, rebasing onto master seems to have regressed the performance improvements. Okay, I applied the trace-inspector-section-latency patch and it still prints out time values well below 1ms.

[DmitryArefiev]

@bluebear94 @cbjeukendrup Ready for testing?

[bluebear94]

Should be.

[DmitryArefiev]

Good.

This issue #24863 (comment) no longer occurs on macOS

Tested on Win10, Mac13.6, LinuxUbuntu24.04.1 LTS - PASS

[RomanPudashkin]

@bluebear94 thank you so much!