fix(time signatures): adding a time signature does not deselect current measure(s)

[brobrobrob]

Resolves: #9584

• Change NotationInteraction::addTimeSignature to re-select the current measure after running cmdAddTimeSignature

• More descriptive variable nomenclature

• [x] I signed the CLA

• [x] The title of the PR describes the problem it addresses

• [x] Each commit's message describes its purpose and effects, and references the issue it resolves

• [x] If changes are extensive, there is a sequence of easily reviewable commits

• [x] The code in the PR follows the coding rules

• [x] 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)

[Jojo-Schmitz]

Isn't this a duplicate of #24892 (or vice versa)?

[brobrobrob]

Isn't this a duplicate of #24892 (or vice versa)?

Yes, this is the correct one because it merges into master

[brobrobrob]

This is a draft btw, you don't have to review now! I thought the convention was to mark as 'ready for review' when I am ready for review @Jojo-Schmitz

[Jojo-Schmitz]

Those merge commits should not be there, please try a git pull --rebase upstream master followed by a git push -f

[brobrobrob]

Those merge commits should not be there, please try a git pull --rebase upstream master followed by a git push -f

Done