Doc: NR - Issue 4761 - changing-defaults.itely: Various improvements

Doc: NR - Issue 4761 - changing-defaults.itely: Various improvements

A variety of nitpicks and
improvements reported by
Federico Bruni

Some spelling corrections
and typos fixed.

The section on \set vs.
\override has been
edited in an attmpt to
simplify the explanation
for the reader.

Fixed a couple of TODOs.

Added a @c that references
an open issue to improve
the example for pure/unpure
containers as the example no
longer illustrates what it
used to as of LP 2.17.x +

[fedelogy, 2016-02-20 09:13:44]

just a doubt about the use of @code in one sentence

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

https://codereview.appspot.com/290270043/diff/1/Documentation/notation/changi...
Documentation/notation/changing-defaults.itely:3493: @code{all-suppressed}.  Use
@code{begin-of-line-invisible}
AFAIK all-printed and all-suppressed are not lilypond options, so why putting
them in a @code block? This is also misleading for translators.

[Jean-Charles, 2016-02-20 10:58:53]

Otherwise LGTM

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

https://codereview.appspot.com/290270043/diff/1/Documentation/notation/changi...
Documentation/notation/changing-defaults.itely:2446: time of the context's
creation, properties are initialized from its
from their definitions?
unless idiomatic

[pkx166h, 2016-02-21 11:05:53]

With Jean-Charles M and Federico B's considerations and corrections addressed.

[pkx166h, 2016-02-21 11:06:37]

Thanks for the review.

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

https://codereview.appspot.com/290270043/diff/1/Documentation/notation/changi...
Documentation/notation/changing-defaults.itely:2446: time of the context's
creation, properties are initialized from its
On 2016/02/20 10:58:53, Jean-Charles wrote:
> from their definitions?
> unless idiomatic

'... the context's ...' - singular. I.e. not '... the contexts' ...'

hence 'its' not 'their'.

Unless the initialization is happening 'to' the many definitions of the
properties of the context rather than happening to the one definition of context
itself (and using various properties for that initialization)?

I.e. context has a single definition defined by many properties - at least that
was my interpretation. :)

https://codereview.appspot.com/290270043/diff/1/Documentation/notation/changi...
Documentation/notation/changing-defaults.itely:3493: @code{all-suppressed}.  Use
@code{begin-of-line-invisible}
On 2016/02/20 09:13:43, fedelogy wrote:
> AFAIK all-printed and all-suppressed are not lilypond options, so why putting
> them in a @code block? This is also misleading for translators.

Done.

[pkx166h, 2017-03-12 18:33:38]

author James Lowe pkx166h@gmail.com
Fri, 19 Feb 2016 11:54:25 +0000 (11:54 +0000)
committer James Lowe pkx166h@gmail.com
Sun, 28 Feb 2016 09:18:03 +0000 (09:18 +0000)
commit 4299c96f609f79426a181ffed107d17fa6e1fdbd