@lion/ui 0.11.1 → 0.11.2

package/README.md

@@ -37,14 +37,14 @@
   <a href="https://lion.js.org/blog/">Blog</a>
 </p>
 
-**Lion is a set of highly performant, accessible and flexible Web Components.!**
+**Lion is a set of highly performant, accessible, and flexible Web Components.**
 
 They provide an unopinionated, white-label layer that can be extended to your own layer of components.
 
 - **High Performance:** Focused on great performance in all relevant browsers with a minimal number of dependencies.
 - **Accessibility:** Aimed at compliance with the WCAG 2.2 AA standard to create components that are accessible for everybody.
 - **Flexibility:** Provides solutions through Web Components and JavaScript classes which can be used, adopted and extended to fit all needs.
-- **Modern Code:** Lion is distributes as pure es modules.
+- **Modern Code:** Lion is distributed as pure es modules.
 - **Exposes functions/classes and Web Components:** Ships a functionality in its most appropriate form.
 
 > Note: Our demo examples might look simple and not very stylish. This is on purpose. They are designed to be basic so you can easily add your own styles to them to match your intended design, without dealing with styles that are already there.

package/components/combobox/test/lion-combobox.test.js

@@ -1,4 +1,4 @@
-import { defineCE, expect, fixture, html, unsafeStatic } from '@open-wc/testing';
+import { defineCE, expect, fixture, html, unsafeStatic, waitUntil } from '@open-wc/testing';
 import { Required, Unparseable } from '@lion/ui/form-core.js';
 import { sendKeys } from '@web/test-runner-commands';
 import { LionCombobox } from '@lion/ui/combobox.js';
@@ -48,6 +48,80 @@ async function fruitFixture({ autocomplete, matchMode } = {}) {
 }
 
 describe('lion-combobox', () => {
+  describe('Rendering on API change', () => {
+    it('renders when options changed', async () => {
+      const entries = [
+        {
+          id: '1',
+          label: 'Apple',
+        },
+        {
+          id: '2',
+          label: 'Artichoke',
+        },
+        {
+          id: '3',
+          label: 'Asparagus',
+        },
+        {
+          id: '4',
+          label: 'Banana',
+        },
+        {
+          id: '5',
+          label: 'Pineapple',
+        },
+      ];
+
+      class Wrapper extends LitElement {
+        static properties = {
+          ...super.properties,
+          entries: { type: Array },
+        };
+
+        constructor() {
+          super();
+          this.entries = entries;
+        }
+
+        render() {
+          return html`<lion-combobox name="foo">
+            ${this.entries.map(
+              entry => html`<lion-option .choiceValue="${entry.id}">${entry.label}</lion-option>`,
+            )}
+          </lion-combobox>`;
+        }
+      }
+
+      const tagString = defineCE(Wrapper);
+      const tag = unsafeStatic(tagString);
+      const wrapperElement = /** @type {Wrapper} */ (await fixture(html`<${tag}></${tag}>`));
+      await waitUntil(
+        () =>
+          wrapperElement?.shadowRoot?.querySelector('lion-options')?.querySelectorAll('lion-option')
+            .length === entries.length,
+      );
+
+      wrapperElement.entries = [
+        {
+          id: '4',
+          label: 'Banana',
+        },
+      ];
+
+      await waitUntil(
+        () =>
+          wrapperElement?.shadowRoot?.querySelector('lion-options')?.querySelectorAll('lion-option')
+            .length === wrapperElement.entries.length,
+      );
+
+      const optionElements = wrapperElement.shadowRoot?.querySelectorAll('lion-option');
+      expect(optionElements?.length).to.equal(wrapperElement.entries.length);
+      const optionElement = optionElements?.item(0);
+      expect(optionElement?.textContent).to.equal('Banana');
+    });
+  });
+
   describe('Options visibility', () => {
     it('hides options when text in input node is cleared after typing something by default', async () => {
       const el = /** @type {LionCombobox} */ (

package/components/localize/src/date/parseDate.js

@@ -1,6 +1,7 @@
 import { getDateFormatBasedOnLocale } from './getDateFormatBasedOnLocale.js';
 import { getLocalizeManager } from '../getLocalizeManager.js';
 import { addLeadingZero } from './utils/addLeadingZero.js';
+import { isCorrectYearFormat } from './utils/checkYearFormat.js';
 
 /**
  * @typedef {import('../../types/LocalizeMixinTypes.js').FormatDateOptions} FormatDateOptions
@@ -63,12 +64,16 @@ export function parseDate(dateString, options) {
   }
 
   const [year, month, day] = parsedString.split('/').map(Number);
+  const [yearString] = parsedString.split('/');
+  if (!isCorrectYearFormat(yearString)) {
+    return undefined;
+  }
   let correctedYear = year;
   if (year < 50) {
     correctedYear = 2000 + year;
   }
 
-  const parsedDate = new Date(new Date(correctedYear, month - 1, day));
+  const parsedDate = new Date(correctedYear, month - 1, day);
   // Check if parsedDate is not `Invalid Date` or that the date has changed (e.g. the not existing 31.02.2020)
   if (
     year > 0 &&

package/components/localize/src/date/utils/checkYearFormat.js

@@ -0,0 +1,8 @@
+/**
+ *
+ * @param {string} year
+ * @returns {boolean}
+ */
+export function isCorrectYearFormat(year) {
+  return /^\d{2}(\d{2})?$/g.test(year);
+}

package/components/localize/test/date/parseDate.test.js

@@ -91,4 +91,10 @@ describe('parseDate()', () => {
     expect(parseDate('02.31.2020')).to.equal(undefined); // non existing date
     expect(parseDate('31.12.2020')).to.equal(undefined); // day & month switched places
   });
+
+  it('returns undefined for incorrect year format', () => {
+    // TODO: we need a `> 0` check for day and month, but maybe consider `>= 0` for year?
+    expect(parseDate('12-12-0000')).to.equal(undefined);
+    expect(parseDate('12-12-999')).to.equal(undefined);
+  });
 });

package/custom-elements.json

@@ -2100,36 +2100,6 @@
         }
       ]
     },
-    {
-      "kind": "javascript-module",
-      "path": "exports/define-helpers/sb-action-logger.js",
-      "declarations": [],
-      "exports": [
-        {
-          "kind": "custom-element-definition",
-          "name": "sb-action-logger",
-          "declaration": {
-            "name": "SbActionLogger",
-            "module": "/exports/helpers.js"
-          }
-        }
-      ]
-    },
-    {
-      "kind": "javascript-module",
-      "path": "exports/define-helpers/sb-locale-switcher.js",
-      "declarations": [],
-      "exports": [
-        {
-          "kind": "custom-element-definition",
-          "name": "sb-locale-switcher",
-          "declaration": {
-            "name": "SbLocaleSwitcher",
-            "module": "/exports/helpers.js"
-          }
-        }
-      ]
-    },
     {
       "kind": "javascript-module",
       "path": "exports/define/lion-accordion.js",
@@ -2820,6 +2790,36 @@
         }
       ]
     },
+    {
+      "kind": "javascript-module",
+      "path": "exports/define-helpers/sb-action-logger.js",
+      "declarations": [],
+      "exports": [
+        {
+          "kind": "custom-element-definition",
+          "name": "sb-action-logger",
+          "declaration": {
+            "name": "SbActionLogger",
+            "module": "/exports/helpers.js"
+          }
+        }
+      ]
+    },
+    {
+      "kind": "javascript-module",
+      "path": "exports/define-helpers/sb-locale-switcher.js",
+      "declarations": [],
+      "exports": [
+        {
+          "kind": "custom-element-definition",
+          "name": "sb-locale-switcher",
+          "declaration": {
+            "name": "SbLocaleSwitcher",
+            "module": "/exports/helpers.js"
+          }
+        }
+      ]
+    },
     {
       "kind": "javascript-module",
       "path": "components/accordion/src/LionAccordion.js",
@@ -145835,6 +145835,39 @@
         }
       ]
     },
+    {
+      "kind": "javascript-module",
+      "path": "components/localize/src/date/utils/checkYearFormat.js",
+      "declarations": [
+        {
+          "kind": "function",
+          "name": "isCorrectYearFormat",
+          "parameters": [
+            {
+              "name": "year",
+              "type": {
+                "text": "string"
+              }
+            }
+          ],
+          "return": {
+            "type": {
+              "text": "boolean"
+            }
+          }
+        }
+      ],
+      "exports": [
+        {
+          "kind": "js",
+          "name": "isCorrectYearFormat",
+          "declaration": {
+            "name": "isCorrectYearFormat",
+            "module": "components/localize/src/date/utils/checkYearFormat.js"
+          }
+        }
+      ]
+    },
     {
       "kind": "javascript-module",
       "path": "../../node_modules/lit/directives/until.js",

package/dist-types/components/localize/src/date/parseDate.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"parseDate.d.ts","sourceRoot":"","sources":["../../../../../components/localize/src/date/parseDate.js"],"names":[],"mappings":"AA4BA;;;;;;GAMG;AACH,sCAJW,MAAM,wFAEJ,IAAI,GAAG,SAAS,CAiD5B;gCA7EY,OAAO,mCAAmC,EAAE,iBAAiB"}
+{"version":3,"file":"parseDate.d.ts","sourceRoot":"","sources":["../../../../../components/localize/src/date/parseDate.js"],"names":[],"mappings":"AA6BA;;;;;;GAMG;AACH,sCAJW,MAAM,wFAEJ,IAAI,GAAG,SAAS,CAqD5B;gCAjFY,OAAO,mCAAmC,EAAE,iBAAiB"}

package/dist-types/components/localize/src/date/utils/checkYearFormat.d.ts

@@ -0,0 +1,7 @@
+/**
+ *
+ * @param {string} year
+ * @returns {boolean}
+ */
+export function isCorrectYearFormat(year: string): boolean;
+//# sourceMappingURL=checkYearFormat.d.ts.map

package/dist-types/components/localize/src/date/utils/checkYearFormat.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"checkYearFormat.d.ts","sourceRoot":"","sources":["../../../../../../components/localize/src/date/utils/checkYearFormat.js"],"names":[],"mappings":"AAAA;;;;GAIG;AACH,0CAHW,MAAM,GACJ,OAAO,CAInB"}