[Libreoffice-commits] core.git: Branch 'libreoffice-7-6' - sw/source

[Balazs Varga (via logerrit)]

 sw/source/core/access/AccessibilityCheck.cxx |   17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

New commits:
commit af32f4f88be29b4b16e0c5b03cb2ea1075aa8e9b
Author:     Balazs Varga <balazs.varga.ext...@allotropia.de>
AuthorDate: Thu Jun 8 11:33:13 2023 +0200
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Mon Jun 12 14:00:09 2023 +0200

    tdf#154943 - Accessibility checker: fix no warn of decorative-flagged image
    
    If an image marked as Decorative it will be irrelevant for a11y,
    and no warning will be on the accessibility sidebar.
    
    Change-Id: If4d25fd8043549832a70b2c2be5aa754c76d9c33
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/152737
    Tested-by: Jenkins
    Reviewed-by: Samuel Mehrbrodt <samuel.mehrbr...@allotropia.de>
    (cherry picked from commit fe7d95eb051312e3174b71be84c122c1837c1f0f)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/152877
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>

diff --git a/sw/source/core/access/AccessibilityCheck.cxx 
b/sw/source/core/access/AccessibilityCheck.cxx
index 250f1b45f901..302d26c9c06f 100644
--- a/sw/source/core/access/AccessibilityCheck.cxx
+++ b/sw/source/core/access/AccessibilityCheck.cxx
@@ -114,12 +114,17 @@ class NoTextNodeAltTextCheck : public NodeCheck
         }
         else if (pNoTextNode->IsGrfNode())
         {
-            auto pIssue = lclAddIssue(m_rIssueCollection, sIssueText,
-                                      
sfx::AccessibilityIssueID::NO_ALT_GRAPHIC);
-            pIssue->setDoc(pNoTextNode->GetDoc());
-            pIssue->setIssueObject(IssueObject::GRAPHIC);
-            pIssue->setObjectID(pNoTextNode->GetFlyFormat()->GetName());
-            pIssue->setNode(pNoTextNode);
+            const SwFrameFormat* pFrameFormat = pNoTextNode->GetFlyFormat();
+            const SfxBoolItem* pIsDecorItem = 
pFrameFormat->GetItemIfSet(RES_DECORATIVE);
+            if (!(pIsDecorItem && pIsDecorItem->GetValue()))
+            {
+                auto pIssue = lclAddIssue(m_rIssueCollection, sIssueText,
+                                          
sfx::AccessibilityIssueID::NO_ALT_GRAPHIC);
+                pIssue->setDoc(pNoTextNode->GetDoc());
+                pIssue->setIssueObject(IssueObject::GRAPHIC);
+                pIssue->setObjectID(pNoTextNode->GetFlyFormat()->GetName());
+                pIssue->setNode(pNoTextNode);
+            }
         }
     }