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?

Compile the .lyx file I attached in the first email of the thread, and
compare the compiled output to LyX output and you should see a
difference. The patch addresses this difference.

Scott

Attachment: signature.asc
Description: PGP signature

Reply via email to