[Libreoffice-commits] core.git: 2 commits - sw/source

[Balazs Varga (via logerrit)]

 sw/source/core/access/AccessibilityCheck.cxx |   18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

New commits:
commit 61845b3d9fed6a8b97e90ec12126d1e1e4cc18d0
Author:     Balazs Varga <balazs.varga.ext...@allotropia.de>
AuthorDate: Wed Aug 2 14:10:28 2023 +0200
Commit:     Balazs Varga <balazs.varga.ext...@allotropia.de>
CommitDate: Thu Aug 3 19:40:27 2023 +0200

    tdf#156570 - A11Y - Fix heading accessibility warning not clickable
    
    Make "Wrong heading..." warning message clickable.
    
    Change-Id: I90d2100551de523e9825cee8788d1c8bb914216c
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155252
    Tested-by: Jenkins
    Reviewed-by: Balazs Varga <balazs.varga.ext...@allotropia.de>

diff --git a/sw/source/core/access/AccessibilityCheck.cxx 
b/sw/source/core/access/AccessibilityCheck.cxx
index 44427dde9ddd..80aef2c4123f 100644
--- a/sw/source/core/access/AccessibilityCheck.cxx
+++ b/sw/source/core/access/AccessibilityCheck.cxx
@@ -1247,7 +1247,10 @@ public:
             }
             resultString
                 = resultString.replaceAll("%LEVEL_CURRENT%", 
OUString::number(currentLevel));
-            lclAddIssue(m_rIssueCollection, resultString);
+            auto pIssue = lclAddIssue(m_rIssueCollection, resultString);
+            pIssue->setIssueObject(IssueObject::TEXT);
+            pIssue->setDoc(pCurrent->GetDoc());
+            pIssue->setNode(pCurrent);
         }
 
         // Updating previous level.
commit 22e8d9c42a2ffa19c9efeeaf581016c39c724134
Author:     Balazs Varga <balazs.varga.ext...@allotropia.de>
AuthorDate: Wed Aug 2 13:05:37 2023 +0200
Commit:     Balazs Varga <balazs.varga.ext...@allotropia.de>
CommitDate: Thu Aug 3 19:40:12 2023 +0200

    tdf#156568 - A11Y - Make clickable the "Hyperlink text is the same as..."
    
    warning message.
    
    Change-Id: Iad391e14f3f414bd4674e634a160bd749b281197
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155225
    Tested-by: Jenkins
    Reviewed-by: Balazs Varga <balazs.varga.ext...@allotropia.de>

diff --git a/sw/source/core/access/AccessibilityCheck.cxx 
b/sw/source/core/access/AccessibilityCheck.cxx
index e4fd6bcc9919..44427dde9ddd 100644
--- a/sw/source/core/access/AccessibilityCheck.cxx
+++ b/sw/source/core/access/AccessibilityCheck.cxx
@@ -332,18 +332,23 @@ private:
         {
             OUString sText = xTextRange->getString();
             INetURLObject aHyperlink(sHyperlink);
+            std::shared_ptr<sw::AccessibilityIssue> pIssue;
             if (aHyperlink.GetProtocol() != INetProtocol::NotValid
                 && INetURLObject(sText) == aHyperlink)
             {
                 OUString sIssueText
                     = 
SwResId(STR_HYPERLINK_TEXT_IS_LINK).replaceFirst("%LINK%", sHyperlink);
-                lclAddIssue(m_rIssueCollection, sIssueText,
-                            sfx::AccessibilityIssueID::HYPERLINK_IS_TEXT);
+                pIssue = lclAddIssue(m_rIssueCollection, sIssueText,
+                                     
sfx::AccessibilityIssueID::HYPERLINK_IS_TEXT);
             }
             else if (sText.getLength() <= 5)
             {
-                auto pIssue = lclAddIssue(m_rIssueCollection, 
SwResId(STR_HYPERLINK_TEXT_IS_SHORT),
-                                          
sfx::AccessibilityIssueID::HYPERLINK_SHORT);
+                pIssue = lclAddIssue(m_rIssueCollection, 
SwResId(STR_HYPERLINK_TEXT_IS_SHORT),
+                                     
sfx::AccessibilityIssueID::HYPERLINK_SHORT);
+            }
+
+            if (pIssue)
+            {
                 pIssue->setIssueObject(IssueObject::TEXT);
                 pIssue->setNode(pTextNode);
                 SwDoc& rDocument = pTextNode->GetDoc();