[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