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

[MU4 Issue] Figured bass - misplaced figures if begins with accidental #11833

Closed
sammik opened this issue May 31, 2022 · 7 comments · Fixed by #18988
Closed

[MU4 Issue] Figured bass - misplaced figures if begins with accidental #11833

sammik opened this issue May 31, 2022 · 7 comments · Fixed by #18988
Assignees
Labels
engraving P2 Priority: Medium

Comments

@sammik
Copy link
Contributor

sammik commented May 31, 2022

Describe the bug
If figured bass beginswith accidental (#, or b) it is horizontaly wrongly placed

To Reproduce
Steps to reproduce the behavior:

  1. Write some notes in one bar
  2. Add figure to first note, press space
  3. Add figure begining with accidental to second note
  4. Figure is placed down

Expected behavior
Figures should be horizontally aligned

Screenshots

figured_bas_bug.mp4

Platform information

  • OS: tested on Linux,but probably platform independent

Additional context
Misplaced only if figures are close each other. If there is extra gap (between bars, ...), figures are placed right.

@Tantacrul
Copy link
Contributor

@oktophonie - what say you?

@Tantacrul Tantacrul added the P3 Priority: Low label Jun 1, 2022
@Tantacrul Tantacrul added this to Needs triage in [MU4.0 BETA1] via automation Jun 1, 2022
@Tantacrul Tantacrul added this to To do in [MU4 - SCORE_VIEW] via automation Jun 1, 2022
@Tantacrul Tantacrul added this to To do in [MU4.0 - ENGRAVING] via automation Jun 1, 2022
@Tantacrul Tantacrul added this to To do in [MU4.0 TEXT] via automation Jun 1, 2022
@oktophonie
Copy link
Contributor

(This is not a regression from 3.6, as the same issue is evident there.)

The problem here (I think) is that the grey line above the figure which shows its duration, and which extends right the way up to the next duration, is not being treated as invisible, and so autoplace is bumping subsequent figures down where they have an accidental at the beginning, because the accidental is placed to the left of the origin point of the string.

image

Note that the first note in the second bar is not bumped down, as the duration line of the previous figure stops at the barline, so autoplace does not kick in.

@sammik
Copy link
Contributor Author

sammik commented Jun 10, 2022

I think, gray duration line should be took into count in autoplace, see first two examples here

figured_bass

So probably accidental in figured bass should be ignored (in autoplace count).

@oktophonie
Copy link
Contributor

It feels backwards (to me) for autoplace to consider an invisible component while ignoring a visible one.
For vertical alignment, aligning figured bass text like lyrics (i.e. consistent along the whole system, though that's configurable via Style -> Autoplace -> Vertical align range) might prove a better solution; other options could then be added, like making sure all figures stay below the lowest note of the system, etc.

@MarcSabatella
Copy link
Contributor

MarcSabatella commented Jun 10, 2022

See the discussion and analysis in the issue that was filed about this for MU3 some time ago - https://musescore.org/en/node/312291

Bottom line - I agree the gray line should probably not be considered when doing autoplace - which is to say, it shouldn't added to the "bbox" for the element. That is definitely what cause the issue, anyhow. But, it does appear to have been done deliberately - see https://github.com/musescore/MuseScore/blob/master/src/engraving/libmscore/figuredbass.cpp#L611-L614

And I think the reason is to avoid this:

image

Not sure it's worth it, though.

Eventually, yes, automatic aligning like is done with chord symbols, and possibly avoiding the lowest note of the system, is the way to go.

@sammik
Copy link
Contributor Author

sammik commented Jun 10, 2022

I just realized, probably we also shouldn't forget continuation lines
figured_bass_cont
(sorry, if it is offtopic)

@Tantacrul Tantacrul removed this from Needs triage in [MU4.0 BETA1] Jun 11, 2022
@Tantacrul Tantacrul added this to To do in 4.x LONGLIST via automation Jun 11, 2022
@Tantacrul Tantacrul removed this from To do in [MU4 - SCORE_VIEW] Jun 11, 2022
@Tantacrul Tantacrul removed this from To do in [MU4.0 - ENGRAVING] Jun 11, 2022
@Tantacrul Tantacrul removed this from To do in [MU4.0 TEXT] Jun 11, 2022
@Tantacrul

This comment was marked as outdated.

4.x LONGLIST automation moved this from To do to Requests Jun 11, 2022
@Tantacrul Tantacrul closed this as not planned Won't fix, can't repro, duplicate, stale Jun 11, 2022
@oktophonie oktophonie changed the title [MU4 Issue] Figured bas - misplaced figures if begins with accidental [MU4 Issue] Figured bass - misplaced figures if begins with accidental Jun 13, 2023
@cbjeukendrup cbjeukendrup moved this from Done to To do in 4.x LONGLIST Jul 23, 2023
@oktophonie oktophonie removed this from To do in 4.x LONGLIST Aug 9, 2023
@oktophonie oktophonie added P2 Priority: Medium engraving and removed P3 Priority: Low labels Aug 9, 2023
@oktophonie oktophonie reopened this Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engraving P2 Priority: Medium
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

5 participants