Merge pull request #110 from momijizukamori/remove-url-defaults

Make URLs slightly cleaner

package.json

@@ -1,6 +1,6 @@
 {
   "name": "bookbinder",
-  "version": "1.3.7",
+  "version": "1.4.0",
   "description": "An app to rearrange PDF pages for printing for bookbinding",
   "type": "module",
   "scripts": {

src/models/configuration.js

@@ -90,12 +90,14 @@ export const schema = z.object({
   bottomEdgePaddingPt: urlSafe(z.coerce.number()).default(0),
   sigFormat,
   sigLength: urlSafe(z.coerce.number()).default(4), // Specific to standard
-  customSigLength: urlSafe(commaSeparatedNumberList).default([]), // Specific to custom.
+  customSigLength: urlSafe(commaSeparatedNumberList).default(null), // Specific to custom.
   foreEdgePaddingPt: urlSafe(z.coerce.number()).default(0), // specific to wacky small
   wackySpacing, // specific to wacky small
   flyleafs: urlSafe(z.coerce.number()).default(1),
-  paperSizeCustomWidth: urlSafe(z.coerce.number()),
-  paperSizeCustomHeight: urlSafe(z.coerce.number()),
+  paperSizeCustomWidth: urlSafe(z.coerce.number()).default(0),
+  paperSizeCustomHeight: urlSafe(z.coerce.number()).default(0),
 });
 
 /** @typedef {z.infer<typeof schema>} Configuration */
+
+export const defaultConfig = schema.parse({});

src/utils/uri.js

@@ -2,6 +2,8 @@
 // License, v. 2.0. If a copy of the MPL was not distributed with this
 // file, You can obtain one at https://mozilla.org/MPL/2.0/.
 
+import { defaultConfig } from '../models/configuration';
+
 /**
  * Gets parameters from a URL.
  * @param { string } url The URL to get the params from
@@ -26,6 +28,11 @@ export const setUrlParams = (url, params) => {
       continue;
     }
 
+    if (value === defaultConfig[key]) {
+      urlRepresentation.searchParams.delete(key);
+      continue;
+    }
+
     urlRepresentation.searchParams.set(key, String(value));
   }