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. rh