anne/lyx_mirror
1
0
Fork 0
mirror of https://git.lyx.org/repos/lyx.git synced 2024-12-22 13:18:28 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Check layout type when inserting inset over selection

Browse Source 
When inserting inset over a selection, the layout is transferred if
the inset covers full paragraphs.

This is not the right policy in the case of environments. Until proper
code is written and tested, limit ourselves to command layouts.

Fixes bug #12251.
This commit is contained in:
Jean-Marc Lasgouttes 2022-01-24 19:19:46 +01:00
parent 6c98ac5a7b
commit 2da183bf4f
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/Text3.cpp
View File

@ -299,8 +299,11 @@ static bool doInsertInset(Cursor & cur, Text * text,
* paragraph and the inset allows setting layout
* FIXME: this does not work as expected when change tracking is on
* However, we do not really know what to do in this case.
* FIXME: figure out a good test in the environment case (see #12251).
*/
if (cur.paragraph().empty() && !inset->forcePlainLayout()) {
if (cur.paragraph().layout().isCommand()
&& cur.paragraph().empty()
&& !inset->forcePlainLayout()) {
cur.paragraph().setPlainOrDefaultLayout(bparams.documentClass());
move_layout = true;
}