From 22e8d9c42a2ffa19c9efeeaf581016c39c724134 Mon Sep 17 00:00:00 2001 From: Balazs Varga Date: Wed, 2 Aug 2023 13:05:37 +0200 Subject: 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 --- sw/source/core/access/AccessibilityCheck.cxx | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) (limited to 'sw') 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 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(); -- cgit