Skip to content

PolylinePlot - additional options - #176

Merged
mathesoncalum merged 2 commits into
musescore:mainfrom
mathesoncalum:polyline_additional_options
Aug 4, 2026
Merged

PolylinePlot - additional options#176
mathesoncalum merged 2 commits into
musescore:mainfrom
mathesoncalum:polyline_additional_options

Conversation

@mathesoncalum

Copy link
Copy Markdown
Contributor

This PR tweaks and adds some additional options to our PolylinePlot component:

  1. Some basic "selected point" logic has been implemented. Eventually this will play a part in keyboard accessibility for polylines (amongst other things). Selected points are disabled by default.
  2. A flag has been introduced to disable ghost points. Ghost points are enabled by default.
  3. "Hovered" is no longer its own style. Instead PolylinePointStyle now contains both the "normal" and "hovered" styles for a given point type. The idea is that, with the introduction of selected points, we should try and keep the number of PolylinePointStyles to a minimum. This means we can simply declare standard, selected, and ghost point styles instead of having to declare standard normal, standard hovered, selected normal, selected hovered, etc.

Build configuration

audacity: audacity/audacity/master
audacity platforms: linux_x64
musescore: musescore/MuseScore/main
musescore platforms: linux_x64

@mathesoncalum
mathesoncalum merged commit 6cdbe6c into musescore:main Aug 4, 2026
2 checks passed
@mathesoncalum
mathesoncalum deleted the polyline_additional_options branch August 5, 2026 08:03
grliszas14 added a commit to audacity/audacity that referenced this pull request Aug 5, 2026
This PR updates `PolylinePlot` usage in accordance with
musescore/muse_framework#176
kryksyh added a commit to audacity/audacity that referenced this pull request Aug 12, 2026
Resolves: backports changes from master to beta 3

List of PRs:

- #11567
- #11522
- #11553
- #11601
- #11606
- #11617
- #11632
- #11638

framework:
- musescore/muse_framework#176
- musescore/muse_framework#210
- musescore/muse_framework#199

deps:
- musescore/muse_deps#40

<!-- Use "x" to fill the checkboxes below like [x] -->

- [x] I signed [CLA](https://www.audacityteam.org/cla/)
- [x] The title of the pull request describes an issue it addresses
- [x] If changes are extensive, then there is a sequence of easily
reviewable commits
- [x] Each commit's message describes its purpose and effects
- [x] There are no behavior changes unnecessary for the stated purpose
of the PR

Recommended:
- [ ] Each commit compiles and runs on my machine without known
undesirable changes of behavior

QA:

- [ ] Testflow test cases have been run
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants