Repository navigation
Insights 6b: lazy sections - rating + forecast, re-solve, pairs & teams, improvement, shelf picks, top row (admins only) - #278
Merged
Merged
Conversation
…s & teams, improvement) WIP: English only Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ster and terms Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…axis + chip fixes - Shelf picks (ShelfPicks) as their own lazy part 'shelf' after the rating frame: likely window through 3c (_shelf_estimate.html.twig, for 6c to replace), share of solvers beaten; free = lock with the count; predictions off = note + setting - Top row: the MSP rating tile reads the stored rating and its place on today's ladder from the context statement (one more column, GetPlayerRatingRanking::sqlRankOf() - the first response stays 10 statements); the 'Try next' and 'MSP rating' finding cards are an eager frame in the findings grid sharing the rating and shelf cache entries - 6a's skill-map axis was cut off on phones: the strips' axis shows from md up; the chip scroller is position: relative (the locked chips' visually-hidden labels widened the page on phones) - 6 locales; tests, budgets and page.md Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…s ShelfPicks - each pick's time = insights/estimate/_line.html.twig (time, certainty bar, tier chip, the shared sheet), resolved for the picks with ResolvePersonalEstimates (members only, flag-gated); the row's picture and name are the links - my own window partial removed; the note explains the ≈ line (6 locales) - budgets: shelf picks ≤ 6 member (4 measured), top row ≤ 16 (14 measured) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… frame errors - pending rows: 'N of 20 solvers · M more needed', the bar against 20 (solversNeeded is what is missing); test at 15 - rating over time: no place in the tooltip (the snapshot's rank follows another rule and not the opt-out); nothing for a private profile; test for opted out + private - pair names in the best results' meta as plain text - escaped once; test with ' and & - insights_frame_controller.js (eager): turbo:frame-missing (prevented - an expired session's sign-in page lands there) and turbo:fetch-request-error show 'Couldn't load this part' + 'Try again' (placeholder back, src anew), aria-busy and role=status cleared; works in the findings grid; 6 locales - rating cache part keyed by the player's privacy and ranking opt-out too - shelf frame budget at route level (7 statements incl. the estimate lines) - rating strips: a visually hidden list with every bar's details Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Insights Phase 6b (
docs/features/insights/implementation-plan.md§6.1, sections 9-11, 13, 15 + the top row): the Insights page's lazy sections. BehindINSIGHTS_PUBLIC; only admins see anything.What changes
No migration.
Budget (cache miss)
Every frame takes about 25-80 ms per request.
Translations
209 new keys, in all 6 locales, with matching placeholders. Register as on the rest of Insights: Czech "vy", French "vous", German "du", Spanish "tú".
🤖 Generated with Claude Code