https://bugs.documentfoundation.org/show_bug.cgi?id=172849

Justin L <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Replace all shows more      |Replace all says it applies
                   |replaced items than there   |the formatting twice, when
                   |are when in end of          |it should only have done it
                   |paragraph                   |once
           See Also|                            |https://bugs.documentfounda
                   |                            |tion.org/show_bug.cgi?id=17
                   |                            |0738
      Regression By|Justin Luth                 |
           Keywords|regression                  |
            Version|27.2.0.0 alpha0+ master     |26.8.0.0 alpha0+ master

--- Comment #3 from Justin L <[email protected]> ---
Since find and replace formatting is not really 'replace', but 'find and
apply', the italic is not supposed to be removed. So there must somehow be two
times when italic is found during a replace-all (assuming the reported number
of replacements is accurate).

Before my patch, if you searched for 'Italic' and replaced with 'Bold Italic'
then 'Search key replaced 2 times." That is effectively what is happening now
with my patch. Thus - not a true regression. 

If you want to really REPLACE the italics with the bold, you would need to
chose the replacement format twice - once with 'no italic' and once with
'bold'.


In OOo and following, LO froze when replacing-all 'Italic' with 'Bold Italic',
until OP's 26.8 commit 17c9b49f3d215b58604a3d301b18ada322e3986f
Author: Mateusz Wlazłowski on Mon Feb 9 10:15:04 2026 +0100
    tdf#131431 Move next node if pCurrentCursor hasn't moved in replaceAll
    ...to avoid an infinite loop
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/185135

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to