sw/source/core/layout/paintfrm.cxx |    2 ++
 1 file changed, 2 insertions(+)

New commits:
commit c51f9f15b8c308e62b4839cc13d623fdab0486c2
Author:     Justin Luth <justin.l...@collabora.com>
AuthorDate: Wed Mar 27 09:50:36 2024 -0400
Commit:     Justin Luth <jl...@mail.com>
CommitDate: Wed Mar 27 18:49:09 2024 +0100

    tdf#131253 sw: draw frame text boundary if fly has textframe
    
    While the text boundary on a wrap-through frame
    will not need text boundary markers for the external text,
    it does have relevance for the internal text.
    
    Change-Id: Ica91dc42333a69cfecd10a72a1188d77d1dd4ed7
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/165395
    Tested-by: Jenkins
    Reviewed-by: Justin Luth <jl...@mail.com>

diff --git a/sw/source/core/layout/paintfrm.cxx 
b/sw/source/core/layout/paintfrm.cxx
index 6903e80b1f46..8479436a88c8 100644
--- a/sw/source/core/layout/paintfrm.cxx
+++ b/sw/source/core/layout/paintfrm.cxx
@@ -7491,8 +7491,10 @@ void SwLayoutFrame::PaintSubsidiaryLines( const 
SwPageFrame *pPage,
             return;
 
         // if the frame is wrap none or wrap through, then text boundary lines 
have no meaning
+        // (unless the frame itself contains text)
         const text::WrapTextMode aSurround = 
GetFormat()->GetSurround().GetSurround();
         if (GetFormat()->GetAnchor().GetAnchorId() != RndStdIds::FLY_AS_CHAR
+            && (!Lower() || !Lower()->IsTextFrame())
             && (aSurround == text::WrapTextMode::WrapTextMode_THROUGH
                 || aSurround == text::WrapTextMode::WrapTextMode_NONE))
         {

Reply via email to