Re: Issue 5490: fix for wrong clefs.varC_change (issue 361800043 by lilyp...@maltemeyn.de)

2019-03-10 Thread innnzzz6

Issue 361800043: Issue 5490: fix for wrong clefs.varC_change

https://codereview.appspot.com/361800043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Re: Issue 5490: fix for wrong clefs.varC_change (issue 361800043 by lilyp...@maltemeyn.de)

2019-03-10 Thread innnzzz6

Issue 361800043: Issue 5490: fix for wrong clefs.varC_change

https://codereview.appspot.com/361800043/

___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel


Issue 5490: fix for wrong clefs.varC_change (issue 361800043 by lilyp...@maltemeyn.de)

2019-03-03 Thread lilypond

Reviewers: ,

Message:
To see the difference between 2.19.82, master and the proposed fix, try

\markup \box \musicglyph #"clefs.varC"
\markup \box \musicglyph #"clefs.varC_change"

Description:
Issue 5490: fix for wrong clefs.varC_change

For solving Issue 5278 I changed the bounding box of the varC clefs
whithout noticing that the height of the vertical bars relies on the box
height. This commit fixes that.

Please review this at https://codereview.appspot.com/361800043/

Affected files (+2, -0 lines):
  M mf/feta-clefs.mf


Index: mf/feta-clefs.mf
diff --git a/mf/feta-clefs.mf b/mf/feta-clefs.mf
index  
b04ff2407d0d06ba27cb042f1b0bb5cc772f8016..de1ec90594b622ec11a9fbd5d33149d83ca79b80  
100644

--- a/mf/feta-clefs.mf
+++ b/mf/feta-clefs.mf
@@ -195,6 +195,8 @@ def draw_varc_clef (expr reduction) =
% make unreduced glyph fit exactly into five staff lines
if reduction = 1:
h := d := 2 staff_space_rounded;
+   else:
+   h := d := 2 reduced_ss;
fi;

% assure that the gap between the left and right stem



___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel