On Tue, Feb 14, 2017 at 10:51:44PM -0500, Richard Heck wrote:
> On 02/14/2017 10:12 PM, Scott Kostyshak wrote:
> > On Mon, Nov 14, 2016 at 04:36:12PM -0500, Scott Kostyshak wrote:
> >> On Sun, Nov 13, 2016 at 07:18:43PM +0000, Guenter Milde wrote:
> >>> On 2016-11-11, Scott Kostyshak wrote:
> >>>
> >>>> [-- Type: text/plain, Encoding:  --]
> >>>> The attached patch fixes the display of the attached .lyx file. When
> >>>> compiled, the .lyx file indents the first line of the environment
> >>>> following the Theorem environment. But LyX shows the line as not
> >>>> indented.
> >>>> There might be similar fixes needed, such as in elsart.layout
> >>>
> >>>
> >>> This might depend on several points, e.g. the "indentfirst" setting of the
> >>> document class or language settings. Some language's typographic tradition
> >>> does not indent the first line after section heading etc. (This may 
> >>> include
> >>> theorems.) 
> >> Agreed. But I think all of these things are taken into consideration
> >> when LyX decides whether to show an indentation. From what I understand,
> >> setting NextNoIndent is just whatever the default class behavior is. And
> >> I believe that behavior is to indent (i.e. NextNoIndent = 0).
> >>
> >>> It may also depend on nesting the theorem inside some paragraph or 
> >>> exporting
> >>> a parbreak (empty line) after the environment or not.
> >>> I don't know the
> >>> current state of the art but I think it should be checked why the current
> >>> setting is as it is.
> >> NextNoIndent was set to 1 at 46611ba1, but the commit message only
> >> states "Fixes to the theorem modules." so I'm not sure how else to
> >> check.
> > Günter (or anyone else), do you object to me committing the patch?
> 
> Please do commit. I find this very annoying and keep meaning to commit
> it myself.

OK, I'll wait a couple of days to see if Günter has an objection. He
might see something that I missed.

Scott

Attachment: signature.asc
Description: PGP signature

Reply via email to