Fix surface colours in the cost comparison table and estimator - #363
Open
gmoranxyz wants to merge 1 commit into
Open
Fix surface colours in the cost comparison table and estimator#363gmoranxyz wants to merge 1 commit into
gmoranxyz wants to merge 1 commit into
Conversation
`--color-card-background-hover` reads like the token set's subtle-surface value, but it is not one: `.gray-section` overrides it to plain white, so on a gray section a header row painted with it comes out lighter than the rows below, and the table stops reading as a card at all. The estimator had the mirror problem: its receipt panel used `card-background`, the same white as the container it sits inside, separated by nothing but a hairline. Add `--color-surface-muted` and `--color-surface-accent`, resolved per section variant so each sits one step off the card surface it lands on, then use them: - the table is a card (`card-background`) with a muted header tint - the Filecoin row takes the brand tint, so it does not merge into the header tint directly above it - mobile cards match: card surface, brand tint on the highlighted one - the estimator column is a card, its receipt panel the inset tint Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017YJ9WRU8X7pPznzQ7k3Q52
|
@claude is attempting to deploy a commit to the FilOz Team on Vercel. A member of the Team first needs to authorize it. |
TippyFlitsUK
approved these changes
Sep 2, 2026
github-project-automation
Bot
moved this from 📌 Triage
to ✔️ Approved by reviewer
in FOC
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
3 participants
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.
--color-card-background-hoverreads like the token set's subtle-surface value, but it is not one:.gray-sectionoverrides it to plain white, so on a gray section a header row painted with it comes out lighter than the rows below, and the table stops reading as a card at all. The estimator had the mirror problem: its receipt panel usedcard-background, the same white as the container it sits inside, separated by nothing but a hairline.Add
--color-surface-mutedand--color-surface-accent, resolved per section variant so each sits one step off the card surface it lands on, then use them:card-background) with a muted header tint