Skip to content

feat(draft): overhaul pick/ban phase with meta tiers, UI pivots, and flex-pick visibility - #326

Open
TtvNekix wants to merge 1 commit into
OpenLeagueManager:developfrom
TtvNekix:feat/draft-phase-improvements
Open

feat(draft): overhaul pick/ban phase with meta tiers, UI pivots, and flex-pick visibility#326
TtvNekix wants to merge 1 commit into
OpenLeagueManager:developfrom
TtvNekix:feat/draft-phase-improvements

Conversation

@TtvNekix

Copy link
Copy Markdown
Contributor

Implements issue #302 — pick/ban phase overhaul.

Changes

1. Meta Tier System (Smart Ban & Pick)

  • New MetaTier enum (S/A/B/C/D) in Rust domain model with scoring
  • Added meta_tier field to Champion and NewChampion structs
  • Parsed from champion-list.json (optional, defaults to "B")
  • Exposed to frontend via ChampionData.meta_tier
  • Draft tip UI shows meta tier badge (colored: red=S, orange=A, blue=B, gray=C/D)

2. UI Pivots (Dynamic Request UI)

  • Reworked tip mixing in ChampionDraft.tsx
  • ALL player pick tips now always visible (one per role, up to 5)
  • When a player changes their request, their bubble updates in-place
  • Other players bubbles stay visible — no more UI blindness
  • During bans: all player ban requests shown first, coach tips after

3. Flex Pick Visibility

  • When a champion has multiple roles (e.g. Corki = MID + ADC), secondary roles rendered as small badges
  • Uses existing roleHints and ROLE_ICON_URLS infrastructure
  • Manager can immediately see flex potential

Files changed: 6 (102 additions, 20 deletions)

…flex-pick visibility

Implements issue OpenLeagueManager#302.

1. Meta tier system: added MetaTier enum (S/A/B/C/D) to champion domain model,
   parsed from champion-list.json or defaulted to B. Exposed to frontend
   via ChampionData.meta_tier. Badge shown on draft suggestion tips.

2. UI pivot fix: reworked tip mixing logic so ALL player tips are always
   visible (one per role). When a player changes their request, other
   players' bubbles no longer get hidden. Pre-slice cap raised to 6-7.

3. Flex-pick visibility: champion suggestion tips now render secondary
   role badges when a champion can flex into other positions (e.g. Corki
   shown as MID + ADC). Uses the existing roleHints/role_icons infra.
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