View Issue Details

IDProjectCategoryView StatusLast Update
0016880ScribusPDFpublic2023-01-04 16:10
Reporterpmjdebruijn Assigned To 
PrioritynormalSeverityminorReproducibilityalways
Status newResolutionopen 
Product Version1.6.0.svn 
Summary0016880: Pre-flight: Raise checkerSettings.minResolution from 144.0 to 200.0
DescriptionSome print on demand services are rejecting PDFs with images with less than 200 DPI images in them.

While such a lower barrier will always be somewhat service specific, I think it's worth considering erring on the higher side by default, therefore see attached patch.
TagsNo tags attached.
PatchYes

Activities

pmjdebruijn

2023-01-04 16:10

reporter  

checker-minresolution.patch (1,986 bytes)   
diff --git a/scribus/prefsmanager.cpp b/scribus/prefsmanager.cpp
index da209c15a..c4a682c07 100644
--- a/scribus/prefsmanager.cpp
+++ b/scribus/prefsmanager.cpp
@@ -2470,7 +2470,7 @@ bool PrefsManager::readPref(const QString& filePath)
 			checkerSettings.checkResolution = static_cast<bool>(dc.attribute("CheckResolution", "1").toInt());
 			checkerSettings.checkPartFilledImageFrames = static_cast<bool>(dc.attribute("CheckPartFilledImageFrames", "0").toInt());
 			checkerSettings.checkTransparency = static_cast<bool>(dc.attribute("CheckTransparency", "1").toInt());
-			checkerSettings.minResolution = ScCLocale::toDoubleC(dc.attribute("MinimumResolution"), 144.0);
+			checkerSettings.minResolution = ScCLocale::toDoubleC(dc.attribute("MinimumResolution"), 200.0);
 			checkerSettings.maxResolution = ScCLocale::toDoubleC(dc.attribute("MaximumResolution"), 4800.0);
 			checkerSettings.checkAnnotations = static_cast<bool>(dc.attribute("CheckAnnotations", "0").toInt());
 			checkerSettings.checkRasterPDF = static_cast<bool>(dc.attribute("CheckRasterPDF", "1").toInt());
@@ -2816,7 +2816,7 @@ void PrefsManager::initDefaultCheckerPrefs(CheckerPrefsList& cp)
 	checkerSettings.checkForGIF = true;
 	checkerSettings.ignoreOffLayers = false;
 	checkerSettings.checkOffConflictLayers = false;
-	checkerSettings.minResolution = 144.0;
+	checkerSettings.minResolution = 200.0;
 	checkerSettings.maxResolution = 2400.0;
 	checkerSettings.checkNotCMYKOrSpot = false;
 	checkerSettings.checkDeviceColorsAndOutputIntent = false;
@@ -2837,7 +2837,7 @@ void PrefsManager::initDefaultCheckerPrefs(CheckerPrefsList& cp)
 	checkerSettings.checkFontIsOpenType = true;
 	checkerSettings.checkTransparency = true;
 	checkerSettings.checkAnnotations = true;
-	checkerSettings.minResolution = 144.0;
+	checkerSettings.minResolution = 200.0;
 	checkerSettings.checkDeviceColorsAndOutputIntent = true;
 	cp.insert(CommonStrings::PDF_X3	, checkerSettings);
 	checkerSettings.checkNotCMYKOrSpot = true;
checker-minresolution.patch (1,986 bytes)   

Issue History

Date Modified Username Field Change
2023-01-04 16:10 pmjdebruijn New Issue
2023-01-04 16:10 pmjdebruijn File Added: checker-minresolution.patch