Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1122)

Issue 337880043: Fix regression introduced with 5122 "Fix not scaling stem ..." (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 years, 4 months ago by thomasmorley651
Modified:
3 years, 11 months ago
Reviewers:
rioujoel
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

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.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+20 lines, -7 lines) Patch
M scm/define-markup-commands.scm View 2 chunks +20 lines, -7 lines 0 comments Download

Messages

Total messages: 2
thomasmorley651
Please review
6 years, 4 months ago (2017-11-05 14:45:43 UTC) #1
rioujoel
6 years, 4 months ago (2017-11-05 15:16:49 UTC) #2
It seems fine to me. This patch corrects the length of metronomic marks in
my scores and I do not see side effects.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b