From 31bafc329705acee4c953852a42e787d08b126d0 Mon Sep 17 00:00:00 2001 From: Veeck Date: Mon, 8 Sep 2025 14:42:13 +0200 Subject: [PATCH] update default icon for calendars (#3879) While looking at https://github.com/MagicMirrorOrg/MagicMirror-Documentation/issues/114 I noticed that the default icon is not named correctly. `calendar-alt` should be called `calendar-days` which seems to be its fallback anyways. I also updated the link to the icon search --------- Co-authored-by: veeck Co-authored-by: Kristjan ESPERANTO <35647502+KristjanESPERANTO@users.noreply.github.com> --- CHANGELOG.md | 1 + modules/default/calendar/calendar.js | 2 +- tests/e2e/modules/calendar_spec.js | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 6653e310..dc440d5b 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -40,6 +40,7 @@ Thanks to: @dathbe. ### Updated +- [calendar] Update defaultSymbol name and also the link to the icon search site (#3879) - [core] Update dependencies including electron to v38 as well as github actions (#3831, #3849, #3857, #3858, #3872, #3876, #3882) - [weather] Update feels_like temperature calculation formula (#3869) diff --git a/modules/default/calendar/calendar.js b/modules/default/calendar/calendar.js index 5c1c9ee7..a1807d22 100644 --- a/modules/default/calendar/calendar.js +++ b/modules/default/calendar/calendar.js @@ -8,7 +8,7 @@ Module.register("calendar", { limitDays: 0, // Limit the number of days shown, 0 = no limit pastDaysCount: 0, displaySymbol: true, - defaultSymbol: "calendar-alt", // Fontawesome Symbol see https://fontawesome.com/cheatsheet?from=io + defaultSymbol: "calendar-days", // Fontawesome Symbol see https://fontawesome.com/search?ic=free&o=r defaultSymbolClassName: "fas fa-fw fa-", showLocation: false, displayRepeatingCountTitle: false, diff --git a/tests/e2e/modules/calendar_spec.js b/tests/e2e/modules/calendar_spec.js index 03886421..56d2a45a 100644 --- a/tests/e2e/modules/calendar_spec.js +++ b/tests/e2e/modules/calendar_spec.js @@ -42,7 +42,7 @@ describe("Calendar module", () => { }); it("should show the default calendar symbol in each event", async () => { - await expect(testElementLength(".calendar .event .fa-calendar-alt", 0, "not")).resolves.toBe(true); + await expect(testElementLength(".calendar .event .fa-calendar-days", 0, "not")).resolves.toBe(true); }); });