sw/source/uibase/utlui/content.cxx | 111 +++++++++----------------------------
1 file changed, 27 insertions(+), 84 deletions(-)
New commits:
commit f8e086f185b31f753074fd22ecc73c44b193a784
Author: Jim Raykowski <rayk...@gmail.com>
AuthorDate: Sun Oct 3 10:15:03 2021 -0800
Commit: Jim Raykowski <rayk...@gmail.com>
CommitDate: Mon Oct 4 07:51:59 2021 +0200
SwNavigator: Use select by address function for tracking Fields,
Comments, and Hyperlinks
Field, comment, and hyperlink tracking selection in the Navigator
content tree is done by comparing content address. This patch uses the
lcl_SelectByContentTypeAndAddress function to replace redundant code.
Change-Id: Idb9f48c46bb432bf2405e4fb0bd8d1830f2c26e5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123031
Tested-by: Jenkins
Reviewed-by: Jim Raykowski <rayk...@gmail.com>
diff --git a/sw/source/uibase/utlui/content.cxx
b/sw/source/uibase/utlui/content.cxx
index f6caf32c5aed..dfb49bc7f6b3 100644
--- a/sw/source/uibase/utlui/content.cxx
+++ b/sw/source/uibase/utlui/content.cxx
@@ -3359,6 +3359,27 @@ static void
lcl_SelectByContentTypeAndAddress(SwContentTree* pThis, weld::TreeVi
p = pCnt->GetTextFootnote();
break;
}
+ case ContentTypeId::URLFIELD:
+ {
+
assert(dynamic_cast<SwURLFieldContent*>(static_cast<SwTypeNumber*>(pUserData)));
+ SwURLFieldContent* pCnt =
static_cast<SwURLFieldContent*>(pUserData);
+ p = static_cast<const SwTextAttr*>(pCnt->GetINetAttr());
+ break;
+ }
+ case ContentTypeId::TEXTFIELD:
+ {
+
assert(dynamic_cast<SwTextFieldContent*>(static_cast<SwTypeNumber*>(pUserData)));
+ SwTextFieldContent* pCnt =
static_cast/*reinterpret_cast*/<SwTextFieldContent*>(pUserData);
+ p = pCnt->GetFormatField()->GetField();
+ break;
+ }
+ case ContentTypeId::POSTIT:
+ {
+
assert(dynamic_cast<SwPostItContent*>(static_cast<SwTypeNumber*>(pUserData)));
+ SwPostItContent* pCnt =
static_cast<SwPostItContent*>(pUserData);
+ p = pCnt->GetPostIt()->GetField();
+ break;
+ }
default:
break;
}
@@ -3564,96 +3585,18 @@ void SwContentTree::UpdateTracking()
// in the tree by name may result in incorrect selection. Find the
item in the tree by
// comparing the SwTextINetFormat pointer at the document cursor
position to that stored
// in the item SwURLFieldContent.
- // find content type entry
- std::unique_ptr<weld::TreeIter> xIter(m_xTreeView->make_iterator());
- bool bFoundEntry = m_xTreeView->get_iter_first(*xIter);
- while (bFoundEntry && SwResId(STR_CONTENT_TYPE_URLFIELD) !=
m_xTreeView->get_text(*xIter))
- bFoundEntry = m_xTreeView->iter_next_sibling(*xIter);
- if (bFoundEntry)
- {
- // assure content type entry is expanded
- m_xTreeView->expand_row(*xIter);
- // find content type content entry and select it
- while (m_xTreeView->iter_next(*xIter) && lcl_IsContent(*xIter,
*m_xTreeView))
- {
- if (aContentAtPos.pFndTextAttr == reinterpret_cast<const
SwURLFieldContent*>
- (m_xTreeView->get_id(*xIter).toInt64())->GetINetAttr())
- {
- // get first selected for comparison
- std::unique_ptr<weld::TreeIter>
xFirstSelected(m_xTreeView->make_iterator());
- if (!m_xTreeView->get_selected(xFirstSelected.get()))
- xFirstSelected.reset();
- if (m_xTreeView->count_selected_rows() != 1 ||
- m_xTreeView->iter_compare(*xIter, *xFirstSelected)
!= 0)
- {
- // unselect all entries and make found entry visible
and selected
- m_xTreeView->set_cursor(*xIter);
- Select();
- }
- break;
- }
- }
- }
+ lcl_SelectByContentTypeAndAddress(this, *m_xTreeView,
ContentTypeId::URLFIELD,
+ aContentAtPos.pFndTextAttr);
return;
}
// fields
if (SwField* pField = m_pActiveShell->GetCurField(); pField &&
!(m_bIsRoot && m_nRootType != ContentTypeId::TEXTFIELD))
{
- OUString sContentType(SwResId(STR_CONTENT_TYPE_TEXTFIELD));
- if (pField->GetTypeId() == SwFieldTypesEnum::Postit)
- sContentType = SwResId(STR_CONTENT_TYPE_POSTIT);
- // find content type entry
- std::unique_ptr<weld::TreeIter> xIter(m_xTreeView->make_iterator());
- bool bFoundEntry = m_xTreeView->get_iter_first(*xIter);
- while (bFoundEntry && sContentType != m_xTreeView->get_text(*xIter))
- bFoundEntry = m_xTreeView->iter_next_sibling(*xIter);
- // find content type content entry and select it
- if (bFoundEntry)
- {
- m_xTreeView->expand_row(*xIter); // assure content type entry is
expanded
- while (m_xTreeView->iter_next(*xIter) && lcl_IsContent(*xIter,
*m_xTreeView))
- {
- if (pField->GetTypeId() == SwFieldTypesEnum::Postit)
- {
- SwPostItContent* pCnt =
reinterpret_cast<SwPostItContent*>(m_xTreeView->get_id(*xIter).toInt64());
- if (pCnt && pField == pCnt->GetPostIt()->GetField())
- {
- // get first selected for comparison
- std::unique_ptr<weld::TreeIter>
xFirstSelected(m_xTreeView->make_iterator());
- if (!m_xTreeView->get_selected(xFirstSelected.get()))
- xFirstSelected.reset();
- if (m_xTreeView->count_selected_rows() != 1 ||
- m_xTreeView->iter_compare(*xIter,
*xFirstSelected) != 0)
- {
- // unselect all entries and make passed entry
visible and selected
- m_xTreeView->set_cursor(*xIter);
- Select();
- }
- break;
- }
- }
- else
- {
- SwTextFieldContent* pCnt =
reinterpret_cast<SwTextFieldContent*>(m_xTreeView->get_id(*xIter).toInt64());
- if (pCnt && pField == pCnt->GetFormatField()->GetField())
- {
- // get first selected for comparison
- std::unique_ptr<weld::TreeIter>
xFirstSelected(m_xTreeView->make_iterator());
- if (!m_xTreeView->get_selected(xFirstSelected.get()))
- xFirstSelected.reset();
- if (m_xTreeView->count_selected_rows() != 1 ||
- m_xTreeView->iter_compare(*xIter,
*xFirstSelected) != 0)
- {
- // unselect all entries and make passed entry
visible and selected
- m_xTreeView->set_cursor(*xIter);
- Select();
- }
- break;
- }
- }
- }
- }
+ ContentTypeId nContentTypeId =
+ pField->GetTypeId() == SwFieldTypesEnum::Postit ?
ContentTypeId::POSTIT :
+
ContentTypeId::TEXTFIELD;
+ lcl_SelectByContentTypeAndAddress(this, *m_xTreeView, nContentTypeId,
pField);
return;
}
// drawing
