Re: NR: Document \remove "Keep_alive_together_engraver" (issue 318580043 by g...@ursliska.de)

2017-02-17 Thread David Kastrup
Urs Liska writes: > Am 17.02.2017 um 09:21 schrieb David Kastrup: >> Urs Liska writes: >> >>> Am 17.02.2017 um 08:34 schrieb d...@gnu.org: Ok, I'll bite. What kind of piano music is written like \score { \new PianoStaff <<

Re: NR: Document \remove "Keep_alive_together_engraver" (issue 318580043 by g...@ursliska.de)

2017-02-17 Thread Simon Albrecht
On 17.02.2017 16:29, Urs Liska wrote: Am 17.02.2017 um 09:21 schrieb David Kastrup: Urs Liska writes: Am 17.02.2017 um 08:34 schrieb d...@gnu.org: Ok, I'll bite. What kind of piano music is written like \score { \new PianoStaff << \new Staff = "up" <<

Re: NR: Document \remove "Keep_alive_together_engraver" (issue 318580043 by g...@ursliska.de)

2017-02-17 Thread Urs Liska
Am 17.02.2017 um 09:21 schrieb David Kastrup: > Urs Liska writes: > >> Am 17.02.2017 um 08:34 schrieb d...@gnu.org: >>> Ok, I'll bite. What kind of piano music is written like >>> >>> \score { >>> \new PianoStaff << >>> \new Staff = "up" << >>> \structure >>>

Re: NR: Document \remove "Keep_alive_together_engraver" (issue 318580043 by g...@ursliska.de)

2017-02-17 Thread Simon Albrecht
On 17.02.2017 09:21, David Kastrup wrote: PianoStaff is explicitly for the case where you want staves to be frenched together. Its name does not mean "Piano inside" but rather "Use frenching conventions common in piano music". And you actually would still not want it to french out multiple

Re: NR: Document \remove "Keep_alive_together_engraver" (issue 318580043 by g...@ursliska.de)

2017-02-17 Thread David Kastrup
Urs Liska writes: > Am 17.02.2017 um 08:34 schrieb d...@gnu.org: >> Ok, I'll bite. What kind of piano music is written like >> >> \score { >> \new PianoStaff << >> \new Staff = "up" << >> \structure >> \v.1 >> \v.2 >> >> >> \dyn.1 >>

Re: NR: Document \remove "Keep_alive_together_engraver" (issue 318580043 by g...@ursliska.de)

2017-02-16 Thread Urs Liska
Am 17.02.2017 um 08:34 schrieb d...@gnu.org: > On 2017/02/17 07:27:27, git wrote: >> On 2017/02/17 07:24:42, dak wrote: > >> > Documentation should not focus about how to use the wrong tool for > the job. >> > If you want to document it, do it the other way round: explain how > you can keep >> >

Re: NR: Document \remove "Keep_alive_together_engraver" (issue 318580043 by g...@ursliska.de)

2017-02-16 Thread dak
On 2017/02/17 07:27:27, git wrote: On 2017/02/17 07:24:42, dak wrote: > Documentation should not focus about how to use the wrong tool for the job. > If you want to document it, do it the other way round: explain how you can keep > the staves of a GrandStaff together and then mention that

Re: NR: Document \remove "Keep_alive_together_engraver" (issue 318580043 by g...@ursliska.de)

2017-02-16 Thread git
On 2017/02/17 07:24:42, dak wrote: https://codereview.appspot.com/318580043/diff/1/Documentation/notation/staff.itely File Documentation/notation/staff.itely (right): https://codereview.appspot.com/318580043/diff/1/Documentation/notation/staff.itely#newcode793

Re: NR: Document \remove "Keep_alive_together_engraver" (issue 318580043 by g...@ursliska.de)

2017-02-16 Thread dak
https://codereview.appspot.com/318580043/diff/1/Documentation/notation/staff.itely File Documentation/notation/staff.itely (right): https://codereview.appspot.com/318580043/diff/1/Documentation/notation/staff.itely#newcode793 Documentation/notation/staff.itely:793: removed from the

Re: NR: Document \remove "Keep_alive_together_engraver" (issue 318580043 by g...@ursliska.de)

2017-02-16 Thread git
Reviewers: lemzwerg, Message: On 2017/02/17 05:38:44, lemzwerg wrote: ... Please use two spaces after a full stop that closes a sentence. Sorry, missed that one. Fixed but without new patch set. Description: NR: Document \remove "Keep_alive_together_engraver" Please review this at

NR: Document \remove "Keep_alive_together_engraver" (issue 318580043 by g...@ursliska.de)

2017-02-16 Thread lemzwerg
LGTM https://codereview.appspot.com/318580043/diff/1/Documentation/notation/staff.itely File Documentation/notation/staff.itely (right): https://codereview.appspot.com/318580043/diff/1/Documentation/notation/staff.itely#newcode790 Documentation/notation/staff.itely:790: empty at the same time.