Re: Issue 347: document that ragged-bottom = #t overrides ragged-last-bottom (issue 257280043 by address@hidden)

[tdanielsmusic]

On 2015/08/11 11:25:23, dak wrote:
> On 2015/08/11 10:48:24, Trevor Daniels wrote:
> >

https://codereview.appspot.com/257280043/diff/1/Documentation/notation/spacing.itely
> > File Documentation/notation/spacing.itely (right):
> >
> >

https://codereview.appspot.com/257280043/diff/1/Documentation/notation/spacing.itely#newcode383
> > Documentation/notation/spacing.itely:383: @code{ragged-last-bottom}
is set to
> > false.
> > On 2015/08/11 10:41:25, dak wrote:
> > > On 2015/08/11 10:16:35, Trevor Daniels wrote:
> > > > I'm not sure this is an improvement.  I think we want to say,
> > > >
> > > > "This controls the spacing of the last page (or the last page
> > > > in each section created with a @code{\bookpart} block) when (and

> > > > only when) ragged-bottom is false.  If ragged-bottom is true the
> > > > value of ragged-last-bottom is ignored and all pages are
ragged."
> > >
> > > "This controls the spacing" does not say in which manner it
controls the
> > > spacing.
> >
> > OK, so "In the same way as ragged-bottom, this controls ...".

> Don't see that I can come up with a change that could unambiguously be
> considered an improvement, so leaving this one open for someone else
to grab.

I see what you mean.  The original wording is correct, and covers the
point made in Issue 347, although in such a subtle way that at least
one experienced developer failed to understand it.  My suggested wording
simply elaborates the point to make it more explicit.  Not sure it
justifies the extra space though.  Unless there are comments encouraging
me to add this change I'll simply mark Issue 347 as Invalid (with
reasons,
of course.)

Trevor

https://codereview.appspot.com/257280043/