From a834a0251f53a561e450fd3c58059bdcfaad8584 Mon Sep 17 00:00:00 2001 From: Caolán McNamara Date: Tue, 24 Mar 2020 14:51:09 +0000 Subject: 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/+/91169 Tested-by: Jenkins Reviewed-by: Adolfo Jayme Barrientos (cherry picked from commit f8e2fe46fa7cd2df90853b0aca31cfda49a06d3d) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/91395 Reviewed-by: Michael Weghorn --- sd/source/ui/view/DocumentRenderer.cxx | 1 + 1 file changed, 1 insertion(+) (limited to 'sd') diff --git a/sd/source/ui/view/DocumentRenderer.cxx b/sd/source/ui/view/DocumentRenderer.cxx index c8a1456d1c7b..cf0f50e03a3c 100644 --- a/sd/source/ui/view/DocumentRenderer.cxx +++ b/sd/source/ui/view/DocumentRenderer.cxx @@ -617,6 +617,7 @@ namespace { else aBuf.append(','); aBuf.append(OUString::number(pPage->GetPageNum() / 2 + 1)); + aPageRange = aBuf.getStr(); } nPrintRange = 1; } -- cgit