[Libreoffice-commits] core.git: sd/source
Xisco Fauli (via logerrit)
logerrit at kemper.freedesktop.org
Tue Feb 11 21:16:32 UTC 2020
sd/source/ui/slidesorter/controller/SlsSelectionObserver.cxx | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
New commits:
commit 7ed602a3b8c0ffe922b4f082cd4cdaa5a8f0d64c
Author: Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Tue Feb 11 12:11:32 2020 +0100
Commit: Xisco Faulí <xiscofauli at libreoffice.org>
CommitDate: Tue Feb 11 22:15:58 2020 +0100
tdf#130440 only select first page if no other page is selected
Found while implementing the UItest.
See https://gerrit.libreoffice.org/c/core/+/88437
it asserts on line 34 AssertionError: 2 != 1 as the first and
second slides are selected
Change-Id: I249dc0ac6faa55f0f15deedb848beabbc64aeed4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88439
Tested-by: Xisco Faulí <xiscofauli at libreoffice.org>
Reviewed-by: Xisco Faulí <xiscofauli at libreoffice.org>
diff --git a/sd/source/ui/slidesorter/controller/SlsSelectionObserver.cxx b/sd/source/ui/slidesorter/controller/SlsSelectionObserver.cxx
index 381933c234a5..5ac6fddda3f4 100644
--- a/sd/source/ui/slidesorter/controller/SlsSelectionObserver.cxx
+++ b/sd/source/ui/slidesorter/controller/SlsSelectionObserver.cxx
@@ -126,7 +126,7 @@ void SelectionObserver::EndObservation()
bool bSuccess = mrSlideSorter.GetController().GetFocusManager().SetFocusedPageToCurrentPage();
// tdf#129346 nothing currently selected, select something, if possible
// but (tdf#129346) only if setting focus to current page failed
- if (!bSuccess && rSelector.GetPageCount())
+ if (!bSuccess && rSelector.GetPageCount() && rSelector.GetSelectedPageCount() == 0)
rSelector.SelectPage(0);
}
More information about the Libreoffice-commits
mailing list