Re: Changes.tely updated - 2.19.x up to September 2014 (issue 147860043 by address@hidden)

[dak]

https://codereview.appspot.com/147860043/diff/1/Documentation/changes.tely
File Documentation/changes.tely (right):

https://codereview.appspot.com/147860043/diff/1/Documentation/changes.tely#newcode69
Documentation/changes.tely:69: \once \override ChorName.text = #"foo"
ChordName.text

Maybe make this @lilypond?  If you keep it "example", you'll also need
to replace { with @{ and } with @}

https://codereview.appspot.com/147860043/diff/1/Documentation/changes.tely#newcode72
Documentation/changes.tely:72: @example
More like @end example

https://codereview.appspot.com/147860043/diff/1/Documentation/changes.tely#newcode75
Documentation/changes.tely:75: with @code{DymanicTexts} or
@code{LyricTexts}.
I think you want all of those without the final s.  And it's DynamicText
rather than DymanicText.

https://codereview.appspot.com/147860043/diff/1/Documentation/changes.tely#newcode102
Documentation/changes.tely:102: fonts other than the default Ementaler.
Emmentaler.  I'm not sure this makes sense without example or reference.

https://codereview.appspot.com/147860043/diff/1/Documentation/changes.tely#newcode106
Documentation/changes.tely:106: grob can now be aligned on the
@q{center} of its parent.
Is this properly documented anywhere?  If not, it's a bit hearsay
without specifying at least that this goes via parent-alignment-X and
parent-alignment-Y (if I remember correctly).

https://codereview.appspot.com/147860043/diff/1/Documentation/changes.tely#newcode108
Documentation/changes.tely:108: @item Improvements to the
@code{\partial} command have been made to
Bar checks are not a "parallel context", but since this is really more a
bug fix than anything else, I'd just remove that item rather than haggle
over its wording.

https://codereview.appspot.com/147860043/diff/1/Documentation/changes.tely#newcode113
Documentation/changes.tely:113: aligned seperately rather than simply
centered against the clef glyph.
"separately", but again I think that's more a bug fix than anything
else.

https://codereview.appspot.com/147860043/diff/1/Documentation/changes.tely#newcode120
Documentation/changes.tely:120: @item Improvements made to smob
functionality. Basic smob types
Internals.  Does not belong into Changes.

https://codereview.appspot.com/147860043/diff/1/Documentation/changes.tely#newcode150
Documentation/changes.tely:150: @code{Rest_collision} grobs now ignore
other grobs that do not use the
I think that's again more of a bug fix than anything else.  Basically,
it reduces the number of warnings.

I'd remove it rather than fix the spellings of the grob and interface
names ;)

https://codereview.appspot.com/147860043/