Skip to content

Commit 31bafc3

Browse files
rejasveeckKristjanESPERANTO
authored
update default icon for calendars (MagicMirrorOrg#3879)
While looking at MagicMirrorOrg/MagicMirror-Documentation#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 <[email protected]> Co-authored-by: Kristjan ESPERANTO <[email protected]>
1 parent d277a27 commit 31bafc3

File tree

3 files changed

+3
-2
lines changed

3 files changed

+3
-2
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ Thanks to: @dathbe.
4040

4141
### Updated
4242

43+
- [calendar] Update defaultSymbol name and also the link to the icon search site (#3879)
4344
- [core] Update dependencies including electron to v38 as well as github actions (#3831, #3849, #3857, #3858, #3872, #3876, #3882)
4445
- [weather] Update feels_like temperature calculation formula (#3869)
4546

modules/default/calendar/calendar.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ Module.register("calendar", {
88
limitDays: 0, // Limit the number of days shown, 0 = no limit
99
pastDaysCount: 0,
1010
displaySymbol: true,
11-
defaultSymbol: "calendar-alt", // Fontawesome Symbol see https://fontawesome.com/cheatsheet?from=io
11+
defaultSymbol: "calendar-days", // Fontawesome Symbol see https://fontawesome.com/search?ic=free&o=r
1212
defaultSymbolClassName: "fas fa-fw fa-",
1313
showLocation: false,
1414
displayRepeatingCountTitle: false,

tests/e2e/modules/calendar_spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ describe("Calendar module", () => {
4242
});
4343

4444
it("should show the default calendar symbol in each event", async () => {
45-
await expect(testElementLength(".calendar .event .fa-calendar-alt", 0, "not")).resolves.toBe(true);
45+
await expect(testElementLength(".calendar .event .fa-calendar-days", 0, "not")).resolves.toBe(true);
4646
});
4747
});
4848

0 commit comments

Comments
 (0)