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

Big time signatures #25894

Merged
merged 2 commits into from
Jan 15, 2025
Merged

Big time signatures #25894

merged 2 commits into from
Jan 15, 2025

Conversation

mike-spa
Copy link
Contributor

@mike-spa mike-spa commented Dec 20, 2024

Resolves: #20480
Resolves: #25687

This PR introduces new engraving options for Big Time Signatures, including:

  • Basic choice to put them on All staves (default) - Above staves - Across staves
  • Options to style the numerals using Smufl variants (normal - narrow - sans serif)
  • Options for size, alignment, centering, gap between the numbers, vertical offset.
  • Options for positioning courtesy time signatures against the right margin in the above-stave case.

The new options are available in a new section of the Clef, Time & Key signatures style page (which, by the way, has been completely reimplemented in QML).
image

This PR does not include logic and/or UI for choosing on which staves the large time signatures should appear (they now appear at the default system object staves). That will be done as part of the System Object feature by @RomanPudashkin.

@mike-spa mike-spa force-pushed the bigTimeSigs branch 5 times, most recently from f4cfb3b to 995ffb2 Compare January 13, 2025 10:18
@mike-spa mike-spa changed the title Test Big time signatures Jan 13, 2025
@mike-spa mike-spa marked this pull request as ready for review January 13, 2025 12:52
@mike-spa mike-spa requested a review from cbjeukendrup January 13, 2025 13:01
@oktophonie oktophonie added the vtests This PR produces approved changes to vtest results label Jan 13, 2025
@rettinghaus

This comment was marked as resolved.

@oktophonie
Copy link
Contributor

@rettinghaus No, the placement will be set via the system markings UI which is not quite finished yet.

@sammik

This comment was marked as resolved.

src/notation/view/widgets/editstyle.ui Outdated Show resolved Hide resolved
src/engraving/types/types.h Outdated Show resolved Hide resolved
src/engraving/rendering/score/horizontalspacing.cpp Outdated Show resolved Hide resolved
src/engraving/dom/timesig.cpp Show resolved Hide resolved
src/engraving/dom/timesig.cpp Outdated Show resolved Hide resolved
src/engraving/dom/segment.cpp Outdated Show resolved Hide resolved
@mike-spa mike-spa merged commit e2518bd into musescore:master Jan 15, 2025
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vtests This PR produces approved changes to vtest results
Projects
None yet
5 participants