Re: Doc: avoid implicit \relative; issue 4371 (issue 237340043 by address@hidden)

[k-ohara5a5a]

https://codereview.appspot.com/237340043/diff/110001/Documentation/learning/tweaks.itely
File Documentation/learning/tweaks.itely (right):

https://codereview.appspot.com/237340043/diff/110001/Documentation/learning/tweaks.itely#newcode330
Documentation/learning/tweaks.itely:330: b c }
On 2015/06/22 11:04:44, Trevor Daniels wrote:
>   Are you proposing we change this?

Yes, proposing to consider it.
I put the summary on the issue
https://code.google.com/p/lilypond/issues/detail?id=4371#c8

https://codereview.appspot.com/237340043/diff/110001/Documentation/learning/tweaks.itely#newcode2324
Documentation/learning/tweaks.itely:2324: c4( c^\markup { \tiny \sharp }
d4.) c8 }
On 2015/06/22 11:04:44, Trevor Daniels wrote:
> Again, using | as a prettifying aid to the detriment of its primary
purpose.
> The first one does nothing useful and the last bar is not checked.

Yep.  The |-at-front style is much easier to type consistently, and
compensates for missing indentation at broken bars, so I wanted to see
what people thought about it in the docs. (The initial bar check helps
often enough at the star of sequences in variables.)

For the manual, we can just do careful indentation.

https://codereview.appspot.com/237340043/diff/110001/Documentation/notation/pitches.itely
File Documentation/notation/pitches.itely (right):

https://codereview.appspot.com/237340043/diff/110001/Documentation/notation/pitches.itely#newcode407
Documentation/notation/pitches.itely:407: cis'' cis'' cis''! cis''? c''
c'' c''! c''?
On 2015/06/22 11:04:44, Trevor Daniels wrote:
> Wouldn't \relative be preferable here?  And in most of the examples in
this
> section.

Yes, I thought we might stay in absolute mode when thinking explicitly
about pitches for the first section of Pitches, but the '' are more
distracting than \relative
It should be \relative c'' {...} so that the first pitch octave doesn't
obscure the points of these examples.

https://codereview.appspot.com/237340043/