fix(ipfs2filecoin): estimator beside the table, responsive pricing, hero polish - #1
Merged
gmoranxyz merged 3 commits intoAug 5, 2026
Conversation
Fold the label, textarea and action into a single bordered panel with a live "X of Y lines are CIDs" readout and a Cmd/Ctrl+Enter shortcut. Move the verdict into its own CidListVerdict component with an icon per outcome, and extract pluralize so the readout and verdict agree. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
… responsive Give each pricing row a detail sub-line so the provider is what you compare on and the plan tier stays secondary, and key rows by provider plus tier now that two Pinata tiers share a name. On mobile the three columns do not fit, so each provider becomes a stacked card; the full table returns at md. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
… hero and type Move the estimator into a companion card beside the comparison table (the table is the market rate, the card prices it for your data), with the two inputs side by side and a compact stacked result that fits the narrow column. Add a faint brand glow along the bottom of the hero, step the reason-card and agent-warning copy down a size, and render How it works four across on desktop. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 task
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
2 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.
Summary
Design-review round 2 for
/ipfs2filecoin, layered on top of FilOzone#350. This targetsipfs2filecoin-design-review(notmainorfeat/ipfs2filecoin-landing-page) on purpose, so the diff here is only the new polish and it flows through FilOzone#350 intofeatand thenmainas one piece, the same way FilOzone#350 stacks on FilOzone#346.Three commits, each independently revertable, each typechecks on its own.
Key changes
feat: the CID checker becomes one input surface. The label, textarea and action fold into a single bordered panel with a live "X of Y lines are CIDs" readout and a Cmd/Ctrl+Enter shortcut. The verdict moves into its ownCidListVerdictcomponent with an icon per outcome, andpluralizeis extracted so the readout and verdict agree.fix: pricing table: provider split from tier, and responsive. Each row gains adetailsub-line so the provider is what you compare on and the plan tier stays secondary; rows now key on provider plus tier, since two Pinata tiers share a name (this also clears a duplicate-key React warning). On mobile the three columns do not fit and the egress value is a full sentence, so each provider becomes a stacked card; the full table returns atmd.fix: estimator beside the table, plus hero and type polish. The deposit estimator moves into a companion card beside the comparison table: the table is the market rate, the card prices it for your own data. Its two inputs sit side by side, and the wide three-column result is reworked into a compact stacked receipt that fits the narrow column. The hero gains a faint brand glow (the--color-brand-500token at 12%) along its bottom edge, the reason-card and agent-warning copy step down a size, and "How it works" renders four across on desktop.Testing
npx tsc --noEmitpasses; each commit typechecks independently.biome checkclean.Follow-up (deliberately not in this PR)
The
$2.50rate is still restated as a literal in three places (the rate footnote, one reason card, and the SEO description) rather than referencingUSD_PER_TIB_MONTH_PER_COPY. The comparison table and estimator both derive from the constant, so those three can silently drift. Left out to keep this focused; happy to wire them up as a follow-up.🤖 Generated with Claude Code