Repository navigation
Feat/ids and tokens - #63
Merged
Merged
Conversation
The feeds give account ids in hex while the Accounts list gives bech32, so a reader could not match a row on one page to a row on the other. The routes now add the Guardian's own bech32 spelling beside every hex id (global and per-account deltas, proposals, the transaction detail's notes and recipient), converted server-side with the SDK, and the pages show it. Hex stays in the URLs and under Technical details. The price book keeps every listed token's symbol and decimals, not only the priced ones, and the routes add them beside each faucet id. Amounts read "1.50 IETH" where a list names the token and stay raw with "units" where none does. The account's Assets card prices each holding the way the Overview total does, and the transaction detail names its account. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017aUkMPsKoKHZ21s3YvsYxL
The chart caption already names it; the label beside the figure repeated it. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017aUkMPsKoKHZ21s3YvsYxL
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017aUkMPsKoKHZ21s3YvsYxL
The page is the account, so the word was repeated; the longer Unfreeze label also widened the button after a freeze. The modals keep the object. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017aUkMPsKoKHZ21s3YvsYxL
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017aUkMPsKoKHZ21s3YvsYxL
tailwind-merge files text-label with the text colour classes, so the coloured Unfreeze button lost its font size and rendered taller. The three header buttons now take their size from the variant, like every toolbar. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017aUkMPsKoKHZ21s3YvsYxL
… size tailwind-merge had no idea text-label and friends are font sizes and filed them with the text colours, so "text-label text-state-active" lost the size and the Unfreeze button rendered at the browser default. Declaring the scale once in cn() fixes every such pairing in the app. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017aUkMPsKoKHZ21s3YvsYxL
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
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.
No description provided.