feat(ui): implement blue-accent reference design across all pages - #33
Merged
Conversation
Rebuild every page (dashboard, blocks, transactions, IBC transfers, validators, proposals, accounts, parameters) and the shared shell (sidebar, top nav, connect flow) around a shared blue-accent design system (panel-surface / reference-table-shell / reference-pill), moving off the previous amber/violet system to match the approved reference design while keeping all existing hooks, routing, and RPC behavior intact. Also included, found and fixed during a follow-up UI/UX + WCAG 2.1 AA review pass: - contrast, keyboard-operability, focus-visibility, form-label, skip- link, live-region, and reduced-motion fixes across the app - theme-aware status/type pill styling via a new shared pillStyle utility, replacing three duplicated hardcoded-hex implementations - real validator uptime via the slashing module's signing-info query (previously a permanently-fake "N/A" field) - corrected IBC transfer event parsing (channel/amount/denom were reading the wrong event attribute keys) with proper ticker conversion - mobile layout fixes (top bar search/connection controls were hidden below breakpoint with no alternative, avatar badges were losing their aspect ratio under flex pressure) - removed dead code (unused Datatable/SearchBar/CodeBlock components)
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
- BlockDetail: App Hash and Proposer were always blank because getBlock() returned StargateClient's narrow BlockHeader type, which omits appHash/proposerAddress even though the raw Tendermint response always carries them. Now merges the real bytes in from the raw block query. - CopyText nested inside a Link (Transactions.tsx's tx hash cell) created two overlapping interactive elements: clicking the copy icon both copied the hash AND navigated to the tx detail page, and keyboard users got a redundant second tab stop. CopyText now accepts an overridable tabIndex (default 0) and calls preventDefault() on copy, so nested usages can opt out of the tab stop and the click no longer bubbles into the parent link's navigation.
Cleanup pass across packages, code, and docs: - Remove unused dependencies: @tanstack/react-table and lucide-react (zero imports anywhere in src/), react-syntax-highlighter and its @types package (orphaned when CodeBlock was deleted), @types/jest (project uses Vitest's own globals, not Jest) - Delete dead files: components/Empty.tsx and hooks/useDebounce.ts (no consumers), encoding/proposal.ts and its test (decoded the legacy gov v1beta1 TextProposal content; this app reads gov v1's proposal.summary directly and never called it) - Trim unused exports from utils/helper.ts (displayCoin, normalizeUrl, getUrlFromPath, and an unused CSV/JSON export feature: convertToCSV, downloadData, exportTransactionsToCSV/JSON, exportBlocksToCSV/JSON) and utils/constant.ts (GOV_PARAMS_TYPE) - Update README to drop references to deleted components/dependencies, correct the i18n claim, and document IBC Transfers, real validator uptime, and the WCAG 2.1 AA accessibility work as features Verified: lint, typecheck, full test suite, and production build all pass; smoke-tested live against a connected RPC.
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
1 participant
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.
Summary
pillStyleutility (previously duplicated 3x with hardcoded hex); real validator uptime via the slashing module's signing-info query (was a permanently-fake "N/A"); corrected IBC transfer event parsing (channel/amount/denom were reading the wrong event attribute keys); mobile layout fixes (top bar controls hidden below breakpoint with no alternative, avatar badges losing aspect ratio under flex pressure); removed dead code (Datatable,SearchBar,CodeBlock).Test plan
pnpm lint— cleanpnpm exec tsc -b --noEmit— cleanpnpm test -- --run— 10/10 passing