Skip to content

feat: add Active Now indicator and Traveling Spirit event#2

Open
HugeFrog24 wants to merge 1 commit into
Silverfeelin:mainfrom
HugeFrog24:feature/active-now-and-traveling-spirit
Open

feat: add Active Now indicator and Traveling Spirit event#2
HugeFrog24 wants to merge 1 commit into
Silverfeelin:mainfrom
HugeFrog24:feature/active-now-and-traveling-spirit

Conversation

@HugeFrog24
Copy link
Copy Markdown

Summary

This PR introduces a minimal “Active Now” state for recurring events by extending the event data model with an optional duration field. Events with a non‑zero duration display an active indicator (with remaining time) during their runtime. Durations are approximate and intended as a foundation for future tuning.

Additional small UI improvements are included (contributor layout) and a structural hook for special‑case events (e.g. Traveling Spirit).

Scope

  • Adds optional duration to event definitions
  • Derives isActive / remaining time from existing timing logic
  • Displays “Active Now” during active windows
  • Keeps reset events instantaneous (duration: 0)
  • Minor mobile layout fix for contributors
  • Introduces a placeholder component for special events

Testing

  • Verified event states across cycles by advancing system time
  • Confirmed that active windows transition correctly to next-event countdown
  • Checked reset events never enter an active state
  • Manually tested layout changes via responsive viewport
  • No errors reported from linter & console

Durations are intentionally conservative and data-driven; values can be adjusted independently without changing logic.

- Show "Active Now (Xm left)" for events that are currently active
- Add duration property to all events for active state calculation
- Add Traveling Spirit bi-weekly event (Thu-Sun every 2 weeks)
- Center contributors section on both mobile and desktop
- Rename "Concert Starts" to "Concert"
- Fix missing key prop warning in Contributors component
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