Skip to content

feat: add Div Performance tab for division session stats#36

Open
michaelrstewart1 wants to merge 1 commit into
landaire:mainfrom
michaelrstewart1:feat/div-performance-tab
Open

feat: add Div Performance tab for division session stats#36
michaelrstewart1 wants to merge 1 commit into
landaire:mainfrom
michaelrstewart1:feat/div-performance-tab

Conversation

@michaelrstewart1

Copy link
Copy Markdown

Summary

Adds a Div Performance chart tab to the Stats section for tracking division/clan battle session performance over time.

What it does

  • Line chart showing each division member's stat plotted per game over a session
  • Supported stats: Damage, Spotting Damage, Frags, Raw XP, Base XP, Win Rate, Personal Rating (PR)
  • Local player always included as the first series (sourced from \PerGameStat\ fields, labeled with their username)
  • Distinct color palette: 10 hand-picked colors indexed by player position — no more ambiguous similar hues
  • Rolling average and per-game labels toggles (same as the Charts tabs)
  • Player toggle controls to show/hide individual players
  • Tab layout: Overview alone on the left panel; Div Performance + Charts tabs share the right panel

DB changes

Two auto-applied migrations:

  • \

Adds a new 'Div Performance' chart tab to the Stats section that plots
each division member's performance over a session of games (primarily
for clan battles / ranked).

Changes:
- New ui/team_performance.rs: line chart showing per-player stat over
  time with rolling average, player toggle controls, and a distinct
  10-color palette for contrast
- Supported stats: Damage, Spotting Damage, Frags, Raw XP, Base XP,
  Win Rate, Personal Rating (PR)
- The local player is always included as the first series, sourced
  from PerGameStat fields and labeled with their username
- Tab layout: Overview alone on the left; Div Performance + Charts tabs
  on the right panel
- DB migration 005: team_members JSON column on session_stats
- DB migration 006: player_name column on session_stats
- Fix: replaced deprecated closeable() with is_closeable() and
  corrected OnCloseResponse variant (was DoNotClose, now Ignore)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@michaelrstewart1

Copy link
Copy Markdown
Author
image

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