[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5231 Fix regression introduced with 5122 "Fix not scaling stem ..."

[Auto mailings of changes to Lily Issues via Testlilyissues-auto]

Unfortunately it seems the stem scaling is still not correct in all cases:

\version "2.19.82"
\book {
  \paper { #(layout-set-staff-size 40) }
  { \tempo 8=88 c'8 }
}

The stem in the markup is twice as large (horizontally and vertically) as it should be. While fiddling around with that example I also noticed that set-global-staff-size has the opposite effect, making the stem too small if you set a size larger than 20:

\version "2.19.82"
\book {
  #(set-global-staff-size 40)
  { \tempo 8=88 c'8 }
}

set-global-staff-size doesn't work inside \book anyway so I suppose it doesn't matter, but I'd still expect the stem size to be consistent with everything else in this example if it was being calculated correctly.

Since everything else, including note heads and flags, staff and ledger lines, stems of ordinary non-markup notes, etc., scales consistently, I'd think there must be already working code somewhere that could be copy-pasted into note-by-number.

---

[issues:#5231] Fix regression introduced with 5122 "Fix not scaling stem ..."

Status: Fixed
Labels: Fixed_2_21_0
Created: Sun Nov 05, 2017 02:42 PM UTC by Thomas Morley
Last Updated: Sun Jan 27, 2019 12:28 AM UTC
Owner: Thomas Morley

Fix regression introduced with 5122 "Fix not scaling stem ..."

commit eee677c480c78d58a5215e246575aa94ba2d1897
only took text-font-size from current layout into account.
Now text-font-size from $defaultpaper is respected as well.
As a result note-by-number scales now properly for changes of
set-global-staff-size, layout-set-staff-size and markup-fontsize.

http://codereview.appspot.com/337880043

---

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto