summaryrefslogtreecommitdiff
path: root/fpicker
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2021-10-18 16:28:03 +0100
committerCaolán McNamara <caolanm@redhat.com>2021-10-18 20:29:15 +0200
commit3c168b1dc876e4fca97f73683572462a29227523 (patch)
tree46c5e85379317f1d6fe2553d06228eacc16d6aca /fpicker
parentd989619895a1215d0f701247d5d8368069da8962 (diff)
Resolves: tdf#145169 only position treeview cursor at the first entry
when the view is filled, don't actually select the entry. With these single selection trees the default is to select when the cursor is positioned, so move the unselect to after the cursor positioning to achieve this. Now nothing is considered "selected" in the view unless the user explicitly does that so the contents of the entry are used by default on save/load because nothing is selected in the treeview unless the user does that. Change-Id: I2a740616205bab35614f85f62e8240cd11df90ad Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123754 Tested-by: Caolán McNamara <caolanm@redhat.com> Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'fpicker')
-rw-r--r--fpicker/source/office/fileview.cxx8
1 files changed, 4 insertions, 4 deletions
diff --git a/fpicker/source/office/fileview.cxx b/fpicker/source/office/fileview.cxx
index 788c001a5574..203f1d1d27b3 100644
--- a/fpicker/source/office/fileview.cxx
+++ b/fpicker/source/office/fileview.cxx
@@ -1386,25 +1386,25 @@ void SvtFileView_Impl::ResetCursor()
{
if (mxView->get_visible())
{
- // deselect
- mxView->unselect_all();
std::unique_ptr<weld::TreeIter> xFirst = mxView->make_iterator();
if (mxView->get_iter_first(*xFirst))
{
// set cursor to the first entry
mxView->set_cursor(*xFirst);
}
+ // deselect
+ mxView->unselect_all();
}
else
{
- // deselect
- mxIconView->unselect_all();
std::unique_ptr<weld::TreeIter> xFirst = mxIconView->make_iterator();
if (mxIconView->get_iter_first(*xFirst))
{
// set cursor to the first entry
mxIconView->set_cursor(*xFirst);
}
+ // deselect
+ mxIconView->unselect_all();
}
}