sw/source/core/access/AccessibilityCheck.cxx | 17 +++++++++++++++-- sw/source/core/access/AccessibilityIssue.cxx | 1 + sw/source/core/inc/AccessibilityIssue.hxx | 1 + 3 files changed, 17 insertions(+), 2 deletions(-)
New commits: commit 5a0898d68abac5883e9f431823f5c5fc4c823d23 Author: Balazs Varga <balazs.varga.ext...@allotropia.de> AuthorDate: Thu Jul 6 13:44:10 2023 +0200 Commit: Samuel Mehrbrodt <samuel.mehrbr...@allotropia.de> CommitDate: Mon Jul 10 09:33:48 2023 +0200 tdf#156135 - Add "Go to" action for "Avoid floating text" issue type Change-Id: Ie98f291141f38ac5544c9e51a209fb9bf1f26d1f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/154107 Tested-by: Jenkins Reviewed-by: Samuel Mehrbrodt <samuel.mehrbr...@allotropia.de> (cherry picked from commit b15697c8d5f395bb368e7d96998f2c0b0662b0de) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/154187 diff --git a/sw/source/core/access/AccessibilityCheck.cxx b/sw/source/core/access/AccessibilityCheck.cxx index 8daf85b5276f..165c3b410b95 100644 --- a/sw/source/core/access/AccessibilityCheck.cxx +++ b/sw/source/core/access/AccessibilityCheck.cxx @@ -1125,7 +1125,13 @@ public: const SwNode* startFly = pCurrent->FindFlyStartNode(); if (startFly && startFly->GetFlyFormat()->GetAnchor().GetAnchorId() != RndStdIds::FLY_AS_CHAR) - lclAddIssue(m_rIssueCollection, SwResId(STR_FLOATING_TEXT)); + { + auto pIssue = lclAddIssue(m_rIssueCollection, SwResId(STR_FLOATING_TEXT)); + pIssue->setIssueObject(IssueObject::TEXTFRAME); + pIssue->setObjectID(startFly->GetFlyFormat()->GetName()); + pIssue->setDoc(pCurrent->GetDoc()); + pIssue->setNode(pCurrent); + } } }; @@ -1372,7 +1378,14 @@ void AccessibilityCheck::checkObject(SwNode* pCurrent, SdrObject* pObject) // (Floating objects with text create problems with reading order) if (pObject->HasText() && FindFrameFormat(pObject)->GetAnchor().GetAnchorId() != RndStdIds::FLY_AS_CHAR) - lclAddIssue(m_aIssueCollection, SwResId(STR_FLOATING_TEXT)); + { + auto pIssue = lclAddIssue(m_aIssueCollection, SwResId(STR_FLOATING_TEXT)); + pIssue->setIssueObject(IssueObject::TEXTFRAME); + pIssue->setObjectID(pObject->GetName()); + pIssue->setDoc(*m_pDoc); + if (pCurrent) + pIssue->setNode(pCurrent); + } const SdrObjKind nObjId = pObject->GetObjIdentifier(); const SdrInventor nInv = pObject->GetObjInventor(); diff --git a/sw/source/core/access/AccessibilityIssue.cxx b/sw/source/core/access/AccessibilityIssue.cxx index ebda9a25083b..a9ddc2b75b62 100644 --- a/sw/source/core/access/AccessibilityIssue.cxx +++ b/sw/source/core/access/AccessibilityIssue.cxx @@ -60,6 +60,7 @@ void AccessibilityIssue::gotoIssue() const { case IssueObject::GRAPHIC: case IssueObject::OLE: + case IssueObject::TEXTFRAME: { SwWrtShell* pWrtShell = m_pDoc->GetDocShell()->GetWrtShell(); bool bSelected = pWrtShell->GotoFly(m_sObjectID, FLYCNTTYPE_ALL, true); diff --git a/sw/source/core/inc/AccessibilityIssue.hxx b/sw/source/core/inc/AccessibilityIssue.hxx index 5a6d3e772333..608a41ee2e1f 100644 --- a/sw/source/core/inc/AccessibilityIssue.hxx +++ b/sw/source/core/inc/AccessibilityIssue.hxx @@ -28,6 +28,7 @@ enum class IssueObject TEXT, DOCUMENT_TITLE, FOOTNOTE, + TEXTFRAME, }; class SW_DLLPUBLIC AccessibilityIssue final : public sfx::AccessibilityIssue