sd/source/ui/view/DocumentRenderer.cxx | 1 +
1 file changed, 1 insertion(+)
New commits:
commit c82b891dc9370d4bf72bc07752884603df7d05ac
Author: Caolán McNamara <caol...@redhat.com>
AuthorDate: Tue Mar 24 14:51:09 2020 +0000
Commit: Caolán McNamara <caol...@redhat.com>
CommitDate: Fri Mar 27 10:53:53 2020 +0100
selecting multiple slides and using "print..." doesn't set selection as
range
start impress, copy the default slide a few times, select more than
one slide and use "print..."
"slides" is selected, but just the first selected slide is shown in the
range
since...
commit 76e22667625ce8d16dfaa8617a199b774e987a70
Date: Thu Jun 14 23:29:34 2018 -0300
Configures page range in print dialog
where
aPageRange = aBuf.getStr()
was removed, but if that line is removed then the surrounding loop
of "for (auto pPage: *pPageSelection)" just accumulates text
in the OUStringBuffer to then discard it and no longer makes sense
perhaps this was an oversight rather than a planned change.
Change-Id: I39e76c70714ee39d189016179fb1d13a71ec74ae
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90994
Tested-by: Jenkins
Tested-by: Caolán McNamara <caol...@redhat.com>
Reviewed-by: Caolán McNamara <caol...@redhat.com>
diff --git a/sd/source/ui/view/DocumentRenderer.cxx
b/sd/source/ui/view/DocumentRenderer.cxx
index bb59ccda6b1b..b91771f623c4 100644
--- a/sd/source/ui/view/DocumentRenderer.cxx
+++ b/sd/source/ui/view/DocumentRenderer.cxx
@@ -616,6 +616,7 @@ namespace {
else
aBuf.append(',');
aBuf.append(OUString::number(pPage->GetPageNum() / 2 +
1));
+ aPageRange = aBuf.getStr();
}
nPrintRange = 1;
}
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
