Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add contextual options to toggle alignment/snapping properties #23329

Closed
avvvvve opened this issue Jun 21, 2024 · 0 comments · Fixed by #23447
Closed

Add contextual options to toggle alignment/snapping properties #23329

avvvvve opened this issue Jun 21, 2024 · 0 comments · Fixed by #23447
Assignees
Labels
accessibility Issues related to accessibility task UX/interaction

Comments

@avvvvve
Copy link

avvvvve commented Jun 21, 2024

Context

4.4 introduces new alignment properties for these elements:

  • Hairpins, which have both "Snap to previous" and "Snap to next" toggles
    image

  • Tempo lines, which just has "Snap to next"
    image

Requirement 1

Create two new shortcut-assignable actions in Preferences > Shortcuts:

  • Toggle snap to previous
  • Toggle snap to next

They don't need a default shortcut. These will toggle the properties for both hairpins and tempo lines (for the latter, only "Toggle snap to next").

Requirement 2

Add context menu actions when hairpins and tempo text are right-clicked to control the snapping toggles in properties. The option(s) should be added after "Edit element" and before "Style", with divider lines above/below.

Hairpins:
image

Tempo text:
image

@avvvvve avvvvve added P1 Priority: High UI Visual issues affecting the UI (not notation) UX/interaction labels Jun 21, 2024
@RomanPudashkin RomanPudashkin assigned mathesoncalum and unassigned Eism Jun 25, 2024
@avvvvve avvvvve changed the title Add contextual options to toggle alignment settings on hairpins Add contextual options to toggle alignment/snapping properties Jun 26, 2024
@bkunda bkunda added the accessibility Issues related to accessibility label Jul 1, 2024
@bkunda bkunda added task and removed UI Visual issues affecting the UI (not notation) labels Jul 1, 2024
@bkunda bkunda removed the P1 Priority: High label Jul 1, 2024
RomanPudashkin added a commit that referenced this issue Jul 4, 2024
…_context

Add snap to next/previous to shortcuts and context menus (#23329)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Issues related to accessibility task UX/interaction
Projects
Development

Successfully merging a pull request may close this issue.

4 participants