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
signature.asc
Description: PGP signature