Skip to content

Integrate custom campaign calendar, fix font persistence and resize bugs - #1

Open
elthairselanim wants to merge 4 commits into
gmdima:mainfrom
elthairselanim:main
Open

Integrate custom campaign calendar, fix font persistence and resize bugs#1
elthairselanim wants to merge 4 commits into
gmdima:mainfrom
elthairselanim:main

Conversation

@elthairselanim

Copy link
Copy Markdown

Adapts the timeline to read dates from Foundry's core game.time.calendar (as configured by wgtgm-mini-calendar) instead of assuming the real Gregorian calendar: calendar-aware date storage/picker/display, plus a date format toggle (Month Day, Year vs Day Month, Year).

Also fixes several pre-existing bugs: fonts not persisting when reopening an event for edit (broken string parsing), the timeline canvas never resizing after first render (missing width option), and a dangling Promise when cancelling the event dialog. Fonts now come from Foundry's own Configure Fonts registry instead of a hardcoded list plus an external Google Fonts CDN call. Adds a Delete button for single events, switches "Clear Timeline" to Foundry's own DialogV2.confirm, and trims stale comments/debug console.log noise.

Adapts the timeline to read dates from Foundry's core game.time.calendar
(as configured by wgtgm-mini-calendar) instead of assuming the real
Gregorian calendar: calendar-aware date storage/picker/display, plus a
date format toggle (Month Day, Year vs Day Month, Year).

Also fixes several pre-existing bugs: fonts not persisting when
reopening an event for edit (broken string parsing), the timeline
canvas never resizing after first render (missing width option),
and a dangling Promise when cancelling the event dialog. Fonts now
come from Foundry's own Configure Fonts registry instead of a
hardcoded list plus an external Google Fonts CDN call. Adds a Delete
button for single events, switches "Clear Timeline" to Foundry's own
DialogV2.confirm, and trims stale comments/debug console.log noise.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant