UI overhual#84
Conversation
iForgot321
left a comment
iForgot321
left a comment
There was a problem hiding this comment.
Holy shit the new UI looks crazy good, few small comments on initial look. Haven't dug through the majority of the code yet though, some parts do look very bloated
There was a problem hiding this comment.
There should not be a package.json on the repository root, how is this being used?
| tabIndex={0} | ||
| aria-pressed={isSelected} | ||
| aria-label={`${wind ? `${windName(wind)} wind, ` : ""}${username}, score ${score.toLocaleString()}`} | ||
| sx={{ |
There was a problem hiding this comment.
The CSS here might be a little too much
| sx={{ | ||
| ...frostedButtonBase, | ||
| "@media (hover: hover)": { | ||
| "&:hover": { |
There was a problem hiding this comment.
I think always displaying the dot looks better than only on hover
|
@b006n97 I will take a look at this PR once I have time this week. I will need to set up my local VM to actually inspect everything. May I ask @iForgot321 not to merge this before at least 2 approvals (including the one from me) from reviewers. During this time @b006n97 please read your code thoroughly and make sure you understand every single line. |
|
PR rebased into #97, a more iterative 12-PR review stack. Review the |
3 participants
UI Overhaul
A comprehensive redesign of the frontend, introducing a new theme system, refreshed pages, and reusable design primitives.
Theme & Design System
theme/tokens.ts,theme/index.ts) with scheme-aware palette tokens exposed as--mui-palette-*CSS variables that flip automatically between light and dark.AccentContext, persisting a runtime accent switch.animations.tsand expandedtheme/utils.tshelpers.DESIGN.mddocumenting the design language.New Shared Components
StatCard,IconBadge,SectionHeader.GameSummaryCard,TransactionTypeSelector,StepSection,RoundRequirements, plus a sharedplacement.tshelper.PlacementDistributionBar,StatisticsSummary.Redesigned Pages
hero.webp) and a substantially expanded landing page.Games.tsxoverview, refreshedLiveGames.Navigation & Mobile
NavBarrework (~960 lines changed) with an improved mobile sidebar and broad mobile optimization.Notes for reviewers
backend/prisma/schema.prismashows a large diff but it's purely reformatting (4-space → 2-space indentation) — no schema changes.japaneseGame.service.ts/hongKongGame.service.tschanges are one line each.