@reykjavik/webtools 0.1.18 → 0.1.19

package/CHANGELOG.md

@@ -4,6 +4,16 @@
 
 - ... <!-- Add new lines here. -->
 
+## 0.1.19
+
+_2024-03-11_
+
+- fix: Incorrect alphabetization of accented characters as part of a word …
+  (not just a single character) This fix corrects the sorting of initial
+  letters, but characters inside the string stay mixed in with their
+  unaccented base character.
+- fix: Incorrect `Intl.ListFormat` format in narrow+unit mode
+
 ## 0.1.18
 
 _2024-03-11_

package/README.md

@@ -462,10 +462,11 @@ detection test.)
 
 **`Intl.Collator` and `localeCompare`:**
 
-- When the `sensitivty` option is set to `"base"` or `"accent"`, it will
-  incorrectly treat `ð` and `d` as the same letter, and the acute-accented
-  characters `á`, `é`, `í`, `ó`, `ú` and `ý` get lumped in with their
-  non-accented counterparts.
+- It incorrectly treats `ð` and `d` as the same letter (most of the time), and
+  the acute-accented characters `á`, `é`, `í`, `ó`, `ú` and `ý` get lumped in
+  with their non-accented counterparts (unless the compared).  
+  We fix this only for the first letter in the string, but not for the rest of
+  it.
 
 **`Intl.NumberFormat` and `toLocaleString`:**
 

package/esm/fixIcelandicLocale.privates.js

@@ -1,4 +1,4 @@
-var _a;
+var _a, _b;
 const _Collator = Intl.Collator;
 const mapLocales = (locales) => {
     locales = typeof locales === 'string' ? [locales] : locales || [];
@@ -11,6 +11,8 @@ const mapLocales = (locales) => {
             // or "accent" then `ð` is consiered a variant of `d` and the letters
             // á, é, í, ó, ú, and ý are not treated as separate letters but simply
             // variants of a, e, i, o, u, and y.
+            // Also when the accented characters are part of a word, they are treated
+            // as fully equal to the base letter.
             return ['da'];
         }
         if (_Collator.supportedLocalesOf(loc).length) {
@@ -19,17 +21,35 @@ const mapLocales = (locales) => {
     }
 };
 const combineParts = (parts) => parts.map(({ value }) => value).join('');
-// ===========================================================================
-// Collator
-// ===========================================================================
 const PatchedCollator = function Collator(locales, options) {
-    locales = mapLocales(locales) || locales;
-    const instance = new _Collator(locales, options);
-    Object.setPrototypeOf(instance, PatchedCollator.prototype);
-    return instance;
+    if (!(this instanceof PatchedCollator)) {
+        // @ts-expect-error  (YOLO! Can't be arsed)
+        return new PatchedCollator(locales, options);
+    }
+    const mappedLocales = mapLocales(locales);
+    this.super = _Collator(mappedLocales || locales, options);
+    this.mapped = !!mappedLocales;
+};
+// This is all very hacky, but extending the class *AND* preseving the
+// ability to instantiate without `new` is a bit of a pain.
+// Eagerly interested in finding a better way to do this.
+const CollatorProto = {
+    constructor: PatchedCollator,
+    compare(a, b) {
+        const res1 = this.super.compare(a, b);
+        const a0 = a.charAt(0);
+        const b0 = b.charAt(0);
+        if (/\d/.test(a0 + b0)) {
+            return res1;
+        }
+        const res2 = this.super.compare(a0, b0);
+        return res2 !== 0 ? res2 : res1;
+    },
+    resolvedOptions() {
+        return this.super.resolvedOptions();
+    },
 };
-PatchedCollator.prototype = Object.create(_Collator.prototype);
-PatchedCollator.prototype.constructor = PatchedCollator;
+PatchedCollator.prototype = CollatorProto;
 // Static methods (not patched since "is" is not ACTUALLY supported.)
 PatchedCollator.supportedLocalesOf = _Collator.supportedLocalesOf;
 PatchedCollator.$original = _Collator;
@@ -285,28 +305,29 @@ const _ListFormat = Intl.ListFormat;
 let PatchedListFormat;
 // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
 if (_ListFormat) {
-    PatchedListFormat = class ListFormat extends _ListFormat {
-        constructor(locales, options) {
-            const mappedLocales = mapLocales(locales);
-            super(mappedLocales || locales, options);
-            this.$original = _ListFormat;
-            this.mapped = !!mappedLocales;
-        }
-        format(list) {
-            return this.mapped ? combineParts(this.formatToParts(list)) : super.format(list);
-        }
-        formatToParts(list) {
-            const parts = super.formatToParts(list);
-            if (this.mapped) {
-                for (const item of parts) {
-                    const { value } = item;
-                    if (item.type === 'literal' && (value === ', el.' || value === ', eller')) {
-                        item.value = ', eða';
+    PatchedListFormat = (_b = class ListFormat extends _ListFormat {
+            constructor(locales, options) {
+                const mappedLocales = mapLocales(locales);
+                super(mappedLocales || locales, options);
+                this.mapped = !!mappedLocales;
+            }
+            format(list) {
+                return this.mapped ? combineParts(this.formatToParts(list)) : super.format(list);
+            }
+            formatToParts(list) {
+                const parts = super.formatToParts(list);
+                if (this.mapped) {
+                    for (const item of parts) {
+                        const { value } = item;
+                        if (item.type === 'literal' && (value === ' el. ' || value === ' eller ')) {
+                            item.value = ' eða ';
+                        }
                     }
                 }
+                return parts;
             }
-            return parts;
-        }
-    };
+        },
+        _b.$original = _ListFormat,
+        _b);
 }
 export const _PatchedListFormat = PatchedListFormat;

package/fixIcelandicLocale.privates.js

@@ -1,5 +1,5 @@
 "use strict";
-var _a;
+var _a, _b;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports._PatchedListFormat = exports._PatchedPluralRules = exports._patchedToLocaleDateString = exports._PatchedDateTimeFormat = exports._patchedToLocaleString = exports._PatchedNumberFormat = exports._patchedLocaleCompare = exports._PatchedCollator = void 0;
 const _Collator = Intl.Collator;
@@ -14,6 +14,8 @@ const mapLocales = (locales) => {
             // or "accent" then `ð` is consiered a variant of `d` and the letters
             // á, é, í, ó, ú, and ý are not treated as separate letters but simply
             // variants of a, e, i, o, u, and y.
+            // Also when the accented characters are part of a word, they are treated
+            // as fully equal to the base letter.
             return ['da'];
         }
         if (_Collator.supportedLocalesOf(loc).length) {
@@ -22,17 +24,35 @@ const mapLocales = (locales) => {
     }
 };
 const combineParts = (parts) => parts.map(({ value }) => value).join('');
-// ===========================================================================
-// Collator
-// ===========================================================================
 const PatchedCollator = function Collator(locales, options) {
-    locales = mapLocales(locales) || locales;
-    const instance = new _Collator(locales, options);
-    Object.setPrototypeOf(instance, PatchedCollator.prototype);
-    return instance;
+    if (!(this instanceof PatchedCollator)) {
+        // @ts-expect-error  (YOLO! Can't be arsed)
+        return new PatchedCollator(locales, options);
+    }
+    const mappedLocales = mapLocales(locales);
+    this.super = _Collator(mappedLocales || locales, options);
+    this.mapped = !!mappedLocales;
+};
+// This is all very hacky, but extending the class *AND* preseving the
+// ability to instantiate without `new` is a bit of a pain.
+// Eagerly interested in finding a better way to do this.
+const CollatorProto = {
+    constructor: PatchedCollator,
+    compare(a, b) {
+        const res1 = this.super.compare(a, b);
+        const a0 = a.charAt(0);
+        const b0 = b.charAt(0);
+        if (/\d/.test(a0 + b0)) {
+            return res1;
+        }
+        const res2 = this.super.compare(a0, b0);
+        return res2 !== 0 ? res2 : res1;
+    },
+    resolvedOptions() {
+        return this.super.resolvedOptions();
+    },
 };
-PatchedCollator.prototype = Object.create(_Collator.prototype);
-PatchedCollator.prototype.constructor = PatchedCollator;
+PatchedCollator.prototype = CollatorProto;
 // Static methods (not patched since "is" is not ACTUALLY supported.)
 PatchedCollator.supportedLocalesOf = _Collator.supportedLocalesOf;
 PatchedCollator.$original = _Collator;
@@ -291,28 +311,29 @@ const _ListFormat = Intl.ListFormat;
 let PatchedListFormat;
 // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
 if (_ListFormat) {
-    PatchedListFormat = class ListFormat extends _ListFormat {
-        constructor(locales, options) {
-            const mappedLocales = mapLocales(locales);
-            super(mappedLocales || locales, options);
-            this.$original = _ListFormat;
-            this.mapped = !!mappedLocales;
-        }
-        format(list) {
-            return this.mapped ? combineParts(this.formatToParts(list)) : super.format(list);
-        }
-        formatToParts(list) {
-            const parts = super.formatToParts(list);
-            if (this.mapped) {
-                for (const item of parts) {
-                    const { value } = item;
-                    if (item.type === 'literal' && (value === ', el.' || value === ', eller')) {
-                        item.value = ', eða';
+    PatchedListFormat = (_b = class ListFormat extends _ListFormat {
+            constructor(locales, options) {
+                const mappedLocales = mapLocales(locales);
+                super(mappedLocales || locales, options);
+                this.mapped = !!mappedLocales;
+            }
+            format(list) {
+                return this.mapped ? combineParts(this.formatToParts(list)) : super.format(list);
+            }
+            formatToParts(list) {
+                const parts = super.formatToParts(list);
+                if (this.mapped) {
+                    for (const item of parts) {
+                        const { value } = item;
+                        if (item.type === 'literal' && (value === ' el. ' || value === ' eller ')) {
+                            item.value = ' eða ';
+                        }
                     }
                 }
+                return parts;
             }
-            return parts;
-        }
-    };
+        },
+        _b.$original = _ListFormat,
+        _b);
 }
 exports._PatchedListFormat = PatchedListFormat;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@reykjavik/webtools",
-	"version": "0.1.18",
+	"version": "0.1.19",
 	"description": "Misc. JS/TS helpers used by Reykjavík City's web dev teams.",
 	"main": "index.js",
 	"repository": "ssh://git@github.com:reykjavikcity/webtools.git",