[frogs] tablature: ties and harmonics (issue1669041)

[ Thread Index | Date Index | More lilynet.net/frogs Archives ]







http://codereview.appspot.com/1669041/diff/1/2
File input/regression/tablature-tie-harmonic.ly (right):

http://codereview.appspot.com/1669041/diff/1/2#newcode3
input/regression/tablature-tie-harmonic.ly:3: \header{ texidoc =
"Harmonics in tablature follow the rules for
\header {
  texidoc = "Harmonics in tablature follow the rules for
fret numbers in tie situations."
}

http://codereview.appspot.com/1669041/diff/1/3
File ly/engraver-init.ly (right):

http://codereview.appspot.com/1669041/diff/1/3#newcode748
ly/engraver-init.ly:748: %% the HarmonicParenthesesItem engraver has to
listen to the
HarmonicParenthesesItem grob

http://codereview.appspot.com/1669041/diff/1/3#newcode750
ly/engraver-init.ly:750: \override HarmonicParenthesesItem
#'after-line-breaking =
#harmonic-parentheses-item::apply-tab-note-head-visibility
It'd be simpler to use 'transparent (see my comments concerning
'whiteout vs 'transparent below)

http://codereview.appspot.com/1669041/diff/1/4
File scm/define-grobs.scm (right):

http://codereview.appspot.com/1669041/diff/1/4#newcode871
scm/define-grobs.scm:871: (angularity . 1.5)
These (angularity, white-padding & width) need documenting in
define-grob-properties.scm and adding to an interface in
define-grob-interfaces.scm (either a new interface or
parentheses-interface.)

http://codereview.appspot.com/1669041/diff/1/4#newcode872
scm/define-grobs.scm:872: (half-thickness . 0.05)
This should come from line-thickness.

http://codereview.appspot.com/1669041/diff/1/4#newcode878
scm/define-grobs.scm:878: (width . 0.5)
could be more specific

http://codereview.appspot.com/1669041/diff/1/5
File scm/output-lib.scm (right):

http://codereview.appspot.com/1669041/diff/1/5#newcode505
scm/output-lib.scm:505: (let* ((lp (draw-angled-bracket-stencil LEFT
grob))
indent

http://codereview.appspot.com/1669041/diff/1/6
File scm/tablature.scm (right):

http://codereview.appspot.com/1669041/diff/1/6#newcode146
scm/tablature.scm:146: (let* ((articulations (ly:event-property
(event-cause grob) 'articulations))
let

http://codereview.appspot.com/1669041/diff/1/6#newcode150
scm/tablature.scm:150: (if (eq? 'harmonic-event (ly:event-property art
'class))
(ly:in-event-class? art 'harmonic-event)

http://codereview.appspot.com/1669041/diff/1/6#newcode157
scm/tablature.scm:157: (half-thickness 0.05) ;; urg, hard-coded values
Hmm, tricky.  It'll be difficult (rather convoluted) to get from
TabNoteHead -> HarmonicParenthesesItem.

(not half-thickness though, since this should come from line-thickness
and ly:output-def-lookup)

http://codereview.appspot.com/1669041/diff/1/6#newcode161
scm/tablature.scm:161: (dirp (ly:stencil-aligned-to
could do with a more descriptive name

http://codereview.appspot.com/1669041/diff/1/6#newcode291
scm/tablature.scm:291: (display "\ntransparent: ")(display
tab-note-head-transparent)
remove debug code

http://codereview.appspot.com/1669041/diff/1/6#newcode293
scm/tablature.scm:293: ;; we simply remove the stencil, so we don't have
to care about 'whiteout
I've thought about this for a while, and it's silly that 'whiteout and
'transparent aren't mutually exclusive: if something's transparent, its
whiteout shouldn't be visible.

(see grob.cc)

http://codereview.appspot.com/1669041/show

---
----
Join the Frogs!


Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/