sw/source/core/access/AccessibilityCheck.cxx | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-)
New commits: commit 585b2cec2f6441a698588e5a27d02c14378daf49 Author: Balazs Varga <balazs.varga.ext...@allotropia.de> AuthorDate: Wed Aug 2 13:05:37 2023 +0200 Commit: Michael Stahl <michael.st...@allotropia.de> CommitDate: Mon Aug 7 12:21: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> Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155356 Reviewed-by: Michael Stahl <michael.st...@allotropia.de> diff --git a/sw/source/core/access/AccessibilityCheck.cxx b/sw/source/core/access/AccessibilityCheck.cxx index 2d44e8689b58..80fc5624d7ed 100644 --- a/sw/source/core/access/AccessibilityCheck.cxx +++ b/sw/source/core/access/AccessibilityCheck.cxx @@ -332,18 +332,24 @@ 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();