From 78b157b61eefc36c1c837a724f0cb067fe40f172 Mon Sep 17 00:00:00 2001 From: Thibaut Cuvelier Date: Sun, 8 Oct 2023 01:39:45 +0200 Subject: [PATCH] DocBook: fix closing formatting after deleted text. Previously, when closing font tags, only the previous character's font was used. However, if that character is deleted, it had no change of having the right font tags opened/closed. Hence, look further to compare the font of the current character to output with the font of the previously output character. --- .../export/docbook/tracking_formatting.lyx | 111 ++++++++++++++++++ .../export/docbook/tracking_formatting.xml | 7 ++ src/Paragraph.cpp | 10 +- 3 files changed, 126 insertions(+), 2 deletions(-) create mode 100644 autotests/export/docbook/tracking_formatting.lyx create mode 100644 autotests/export/docbook/tracking_formatting.xml diff --git a/autotests/export/docbook/tracking_formatting.lyx b/autotests/export/docbook/tracking_formatting.lyx new file mode 100644 index 0000000000..44802ff48e --- /dev/null +++ b/autotests/export/docbook/tracking_formatting.lyx @@ -0,0 +1,111 @@ +#LyX 2.4 created this file. For more info see https://www.lyx.org/ +\lyxformat 620 +\begin_document +\begin_header +\save_transient_properties true +\origin unavailable +\textclass article +\use_default_options true +\maintain_unincluded_children no +\language american +\language_package default +\inputencoding utf8 +\fontencoding auto +\font_roman "default" "default" +\font_sans "default" "default" +\font_typewriter "default" "default" +\font_math "auto" "auto" +\font_default_family default +\use_non_tex_fonts false +\font_sc false +\font_roman_osf false +\font_sans_osf false +\font_typewriter_osf false +\font_sf_scale 100 100 +\font_tt_scale 100 100 +\use_microtype false +\use_dash_ligatures true +\graphics default +\default_output_format default +\output_sync 0 +\bibtex_command default +\index_command default +\float_placement class +\float_alignment class +\paperfontsize default +\use_hyperref false +\papersize default +\use_geometry false +\use_package amsmath 1 +\use_package amssymb 1 +\use_package cancel 1 +\use_package esint 1 +\use_package mathdots 1 +\use_package mathtools 1 +\use_package mhchem 1 +\use_package stackrel 1 +\use_package stmaryrd 1 +\use_package undertilde 1 +\cite_engine basic +\cite_engine_type default +\use_bibtopic false +\use_indices false +\paperorientation portrait +\suppress_date false +\justification true +\use_refstyle 1 +\use_formatted_ref 0 +\use_minted 0 +\use_lineno 0 +\index Index +\shortcut idx +\color #008000 +\end_index +\secnumdepth 3 +\tocdepth 3 +\paragraph_separation indent +\paragraph_indentation default +\is_math_indent 0 +\math_numbering_side default +\quotes_style english +\dynamic_quotes 0 +\papercolumns 1 +\papersides 1 +\paperpagestyle default +\tablestyle default +\tracking_changes true +\output_changes false +\change_bars false +\postpone_fragile_content true +\html_math_output 0 +\html_css_as_file 0 +\html_be_strict false +\docbook_table_output 0 +\docbook_mathml_prefix 1 +\author 1075283030 "Thibaut" +\end_header + +\begin_body + +\begin_layout Title +Test: + deleted text just after the end of formatted text +\end_layout + +\begin_layout Standard + +\lang spanish +incluye el archivo +\family typewriter +config.log +\change_deleted 1075283030 1696721001 + +\family default +, + +\change_unchanged + y menciona el compilador que usas. +\end_layout + +\end_body +\end_document diff --git a/autotests/export/docbook/tracking_formatting.xml b/autotests/export/docbook/tracking_formatting.xml new file mode 100644 index 0000000000..fb245c1476 --- /dev/null +++ b/autotests/export/docbook/tracking_formatting.xml @@ -0,0 +1,7 @@ + + +
+Test: deleted text just after the end of formatted text +incluye el archivo config.log y menciona el compilador que usas. +
\ No newline at end of file diff --git a/src/Paragraph.cpp b/src/Paragraph.cpp index 3718bd4a98..333aeb8862 100644 --- a/src/Paragraph.cpp +++ b/src/Paragraph.cpp @@ -3721,10 +3721,16 @@ std::tuple, std::vector, std::vector 0) { + while (last_output_char > 0 && isDeleted(last_output_char)) + --last_output_char; + } FontInfo const font_old = (i == 0 ? (style.labeltype == LABEL_MANUAL ? style.labelfont : style.font) : - getFont(buf.masterBuffer()->params(), i - 1, outerfont).fontInfo()); + getFont(buf.masterBuffer()->params(), last_output_char, outerfont).fontInfo()); Font const font = getFont(buf.masterBuffer()->params(), i, outerfont); tie(tagsToOpen, tagsToClose) = computeDocBookFontSwitch( font_old, font, buf.masterBuffer()->params().fonts_default_family, fs);