Doc: Issue 5034: Expand documentation of mode-changing commands

Doc: Issue 5034: Expand documentation of mode-changing commands

  Add general description of long and short forms and clarify
  wording generally.

  Add description of \lyrics

  Fix typos in chords and percussion

[thomasmorley651, 2017-01-22 18:43:42]

Two nits.

Otherwise LGTM

https://codereview.appspot.com/313380043/diff/1/Documentation/notation/changi...
File Documentation/notation/changing-defaults.itely (right):

https://codereview.appspot.com/313380043/diff/1/Documentation/notation/changi...
Documentation/notation/changing-defaults.itely:2632: into a @code{ChordNames}
context.
Or in FretBoards.
See:
\new FretBoards \chordmode { <c e g c' e'> }

https://codereview.appspot.com/313380043/diff/1/Documentation/notation/changi...
Documentation/notation/changing-defaults.itely:2696: adds an implicit
@code{\lyricsto} command which associates the following
Point to a section where \lyricsto is explained?

[Trevor Daniels, 2017-01-22 21:17:53]

Respond to Harm's comments

[Trevor Daniels, 2017-01-22 21:18:32]

Thanks Harm!

https://codereview.appspot.com/313380043/diff/1/Documentation/notation/changi...
File Documentation/notation/changing-defaults.itely (right):

https://codereview.appspot.com/313380043/diff/1/Documentation/notation/changi...
Documentation/notation/changing-defaults.itely:2632: into a @code{ChordNames}
context.
On 2017/01/22 18:43:42, thomasmorley651 wrote:
> Or in FretBoards.
> See:
> \new FretBoards \chordmode { <c e g c' e'> }

Done.

https://codereview.appspot.com/313380043/diff/1/Documentation/notation/changi...
Documentation/notation/changing-defaults.itely:2696: adds an implicit
@code{\lyricsto} command which associates the following
On 2017/01/22 18:43:42, thomasmorley651 wrote:
> Point to a section where \lyricsto is explained?

Done.

[thomasmorley651, 2017-01-22 22:28:45]

On 2017/01/22 21:18:32, Trevor Daniels wrote:
> Thanks Harm!
> 
>
https://codereview.appspot.com/313380043/diff/1/Documentation/notation/changi...
> File Documentation/notation/changing-defaults.itely (right):
> 
>
https://codereview.appspot.com/313380043/diff/1/Documentation/notation/changi...
> Documentation/notation/changing-defaults.itely:2632: into a @code{ChordNames}
> context.
> On 2017/01/22 18:43:42, thomasmorley651 wrote:
> > Or in FretBoards.
> > See:
> > \new FretBoards \chordmode { <c e g c' e'> }
> 
> Done.
> 
>
https://codereview.appspot.com/313380043/diff/1/Documentation/notation/changi...
> Documentation/notation/changing-defaults.itely:2696: adds an implicit
> @code{\lyricsto} command which associates the following
> On 2017/01/22 18:43:42, thomasmorley651 wrote:
> > Point to a section where \lyricsto is explained?
> 
> Done.

LGTM

Unrelated:

I've noticed this Rietveld-issue is not cc-ed to lilypond-devel_gnu.org.
Doing 
git config -e
I see (among other settings):
[rietveld]
        server = codereview.appspot.com
        cc = lilypond-devel@gnu.org

Is this missing for you?

[t.daniels_treda.co.uk, 2017-01-22 23:02:25]

thomasmorley65@gmail.com wrote Sunday, January 22, 2017 10:28 PM
 
> LGTM

Thanks!

> Unrelated:
> 
> I've noticed this Rietveld-issue is not cc-ed to lilypond-devel_gnu.org.
> Doing
> git config -e
> I see (among other settings):
> [rietveld]
>         server = codereview.appspot.com
>         cc = lilypond-devel@gnu.org
> 
> Is this missing for you?

Good catch - yes.  Now added.

Trevor

[Trevor Daniels, 2017-01-28 21:40:19]

Pushed to Staging as
553410afdd9496f7d9f1498e755ea18de8e70596

Closing ...