diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2020-03-24 14:51:09 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2020-03-27 10:53:53 +0100 |
| commit | c82b891dc9370d4bf72bc07752884603df7d05ac (patch) | |
| tree | 341eecb775fdd35148879950db005c1c373d3a38 /sd | |
| parent | 406ba4f510dd4912acc833c09ab77ceb600fefe8 (diff) | |
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 <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'sd')
| -rw-r--r-- | sd/source/ui/view/DocumentRenderer.cxx | 1 |
1 files changed, 1 insertions, 0 deletions
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; } |