Created attachment 151114 [details]
The attached spreadsheet contains just a diagonal line of "1" spanning over 3 pages.
Open Tools -> Redact.
=> The layout of the data on pages is incorrect. (for another fun result, check with just four "1"s, from A1 to D4)
This is a regression from the following commit. Adding Cc: to Muhammet Kara, please take a look.
author Muhammet Kara <firstname.lastname@example.org> 2019-03-29 12:18:12 +0300
committer Muhammet Kara <email@example.com> 2019-03-29 11:11:58 +0100
Handle different page sizes for redaction
Created attachment 151115 [details]
PDF output from redaction with fresh master build
This one was done with a master build from yesterday, 220.127.116.11.alpha0+ (8f03bdee8225c619305ef210391dcc3b6c6fe284) / Windows 7.
Created attachment 151116 [details]
PDF output from redaction with month old master build
This build is from a 03-28, 18.104.22.168.alpha0+ (ed6a71eafa61bade50219d2ff6233a42ab6d1c17) / Windows 7.
Also note the tiny page number moving across the pages horizontally.
In fact, the content has the correct size, but the page has the incorrect size.
Seems like it is caused by a combination of depending on the content size while determining the page size, and Draw's inability to support a mix of different page sizes.
(In reply to Aron Budea from comment #2)
> Created attachment 151116 [details]
> PDF output from redaction with month old master build
> This build is from a 03-28, 22.214.171.124.alpha0+
> (ed6a71eafa61bade50219d2ff6233a42ab6d1c17) / Windows 7.
> Also note the tiny page number moving across the pages horizontally.
The tiny page number is another issue. Unrelated to this one.
Patch is on Gerrit: https://gerrit.libreoffice.org/#/c/71627/
Muhammet Kara committed a patch related to this issue.
It has been pushed to "master":
tdf#125063: Don't depend on content size for Page sizing
It will be available in 6.3.0.
The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
Affected users are encouraged to test the fix and report feedback.
Verified, thanks for the fix, Muhammet!
Version: 126.96.36.199.alpha0+ (x64)
Build ID: d6e8e41c22023bc15cc4c9659b33c1cc3d6edb75
CPU threads: 4; OS: Windows 6.1; UI render: default; VCL: win;
TinderBox: Win-x86_64@42, Branch:master, Time: 2019-05-06_01:12:47
Locale: en-US (en_US); UI-Language: en-US