View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0016886 | Scribus | public | 2023-01-08 17:13 | 2023-01-24 17:01 | |
Reporter | pmjdebruijn | Assigned To | |||
Priority | normal | Severity | minor | Reproducibility | always |
Status | new | Resolution | open | ||
Product Version | 1.6.0.svn | ||||
Summary | 0016886: PDF Maximum Image Resolution (resampling) isn't enabled by default | ||||
Description | I think RecalcPic, image resampling to (by default) 300dpi should be enabled by default. The vast majority of print shops / print on demand services effectively recommend this. And even for digital (online) publishing, having images with more resolution just means PDFs are slower to download/render. So the use cases where you wouldn't want to resample your images (the current default), seem at best uncommon. | ||||
Tags | No tags attached. | ||||
Patch | Yes | ||||
|
appprefs_recalcpic.patch (580 bytes)
diff -Nurpd a/scribus/prefsmanager.cpp b/scribus/prefsmanager.cpp --- a/scribus/prefsmanager.cpp 2023-01-08 13:38:08.727923073 +0100 +++ b/scribus/prefsmanager.cpp 2023-01-08 12:29:46.000000000 +0100 @@ -458,7 +458,7 @@ void PrefsManager::initDefaults() appPrefs.pdfPrefs.Compress = true; appPrefs.pdfPrefs.CompressMethod = PDFOptions::Compression_Auto; appPrefs.pdfPrefs.Quality = 0; - appPrefs.pdfPrefs.RecalcPic = false; + appPrefs.pdfPrefs.RecalcPic = true; appPrefs.pdfPrefs.embedPDF = false; appPrefs.pdfPrefs.Bookmarks = false; appPrefs.pdfPrefs.PicRes = 300; |
|
Well, I know that InDesign enables resampling by default, but I'd be cautious over enabling image resampling by default in Scribus due to insufficient control on resampling. In one of my former company, we explicitly recommended our customers to disable image resampling settings in InDesign: - image resampling can hurt image quality, especially when resampling for eg a 320dpi image to 300dpi - image resampling is generally to be avoided for 1bit images - raster images containing text usually benefits from resolution over 300dpi, text being a bit fuzzy at 300dpi. Basically we need to implement 0002745 in order to enable resampling for grayscale/color images safely. |
|
wrt... [1] isn't that mostly a matter of resampling algo's? I see PDFLib uses Qt::SmoothTransformation which is in essence bilinear, which explains that, so this isn't really an issue with resampling as a concept, but an issue with the implementation of resampling, something like Lanczos2 would probably be a lot better, and mostly solve this issue. But that won't be a trivial change of course. [2] I guess that makes sense, wouldn't converting to greyscale before resampling mostly solve this? [3] Shouldn't corner cases like that be handled via an additional combo box like an image frame has for PDF Compression where Method/Quality default to Global, but then for Resolution... |
Date Modified | Username | Field | Change |
---|---|---|---|
2023-01-08 17:13 | pmjdebruijn | New Issue | |
2023-01-08 17:13 | pmjdebruijn | File Added: appprefs_recalcpic.patch | |
2023-01-23 20:49 | jghali | Note Added: 0049968 | |
2023-01-23 20:50 | jghali | Note Edited: 0049968 | |
2023-01-23 20:50 | jghali | Note Edited: 0049968 | |
2023-01-24 17:01 | pmjdebruijn | Note Added: 0049971 |