]> git.lyx.org Git - lyx.git/commit
Amend 16a1aa99
authorThibaut Cuvelier <tcuvelier@lyx.org>
Sat, 7 Jan 2023 02:05:55 +0000 (03:05 +0100)
committerThibaut Cuvelier <tcuvelier@lyx.org>
Sat, 7 Jan 2023 02:06:44 +0000 (03:06 +0100)
commit96023697f5efa4d3414bdfeb1dab3200057ba1cb
tree1963e73a102dc15fb2f03b2573a708ef91ce566b
parent3cf8ced72ede681a61918d69ffe2c8d8a1dc4e19
Amend 16a1aa99

With the current settings, a theorem with several paragraphs will have several wrapper tags (i.e. two `<db:figure>`, with one paragraph each, one next to the other when it's the same theorem), while there should be only one (one `<db:figure>` with two paragraphs).

It makes me wonder whether DocBookWrapperMergeWithPrevious should be the default value. I don't see a case where it would be set to `false` with a wrapper tag, but the test suite isn't developed enough to make the change with peace of mind (i.e. remove the parameter `DocBookWrapperMergeWithPrevious` and consider it is always `true` in the code).

For the record, the current code also works with two subsequent theorems with two paragraphs each (see new test case).
autotests/export/docbook/font_overlapping.lyx [new file with mode: 0644]
autotests/export/docbook/theorems-multiparagraph.lyx [new file with mode: 0644]
autotests/export/docbook/theorems-multiparagraph.xml [new file with mode: 0644]
lib/layouts/elsart.layout
lib/layouts/theorems-ams-bytype.inc
lib/layouts/theorems-ams-chap-bytype.inc
lib/layouts/theorems-ams.inc
lib/layouts/theorems-bytype.inc
lib/layouts/theorems-starred.inc
lib/layouts/theorems-without-preamble.inc
lib/layouts/theorems.inc