chore(ensrainbow.io): Update FullRainbow component values and footnote for clarity#2203
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughRefactors BarChart rendering/layout (percent computation, external percent label, rainbow fill width transition) and updates FullRainbow BarChart dataset: "ENS Subgraph" set to 11, "Current" and "Target" bar values/gradients adjusted (Target replaces Future target and includes ENSRainbowBeam), and footnote expanded to note encoded labelhashes for unnormalized names. ChangesENS Rainbow Chart Updates
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR updates the ENSRainbow docs “FullRainbow” chart to reflect revised coverage values and clarifies the explanatory footnote for how results are calculated.
Changes:
- Adjusted the displayed coverage values for “ENS Subgraph”, “ENSRainbow + ENSNode (Current)”, and the target configuration.
- Updated the target label to “ENSRainbow + ENSNode + ENSRainbowBeam” and changed the badge text from “Future target” to “Target”.
- Expanded the footnote to clarify that certain unnormalized/encoded-labelhash formatted names are excluded from results.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Greptile SummaryThis PR refines the "Get the full rainbow" section on ensrainbow.io: the
Confidence Score: 5/5Pure UI and data update with no logic-affecting risk — safe to merge. All changes are confined to visual layout and static data: bar chart layout tweaks, updated coverage percentages and gradient colours, a renamed label, and an extended footnote. The entity in the JSX attribute is correctly decoded to a newline by the JSX parser, and whitespace-pre-wrap renders it as expected. No data flow, API calls, or shared logic is touched. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[FullRainbow] -->|passes data + footnote| B[BarChart]
B --> C{data.map}
C --> D[Compute percent = value/maxValue*100]
D --> E[Render label span above bar row]
E --> F[Flex row: bar container + percent label]
F --> G[Colored bar div at percent% width]
F --> H[Percentage span outside bar, right-aligned]
B --> I{footnote?}
I -->|yes| J[p.whitespace-pre-wrap renders newline from ]
Reviews (8): Last reviewed commit: "Merge branch 'main' into fix-ensrainbow-..." | Re-trigger Greptile |
![vercel[bot]](https://avatars.githubusercontent.com/in/8329?s=60&v=4){kind=small}
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/ensrainbow.io/src/components/organisms/FullRainbow.tsx`:
- Line 44: The footnote prop in FullRainbow is using the literal entity "&`#10`;"
which will not render as a newline; replace that entity with a real newline
character and provide the string as a JS string/template (e.g.
footnote={`Coverage vs. indexing the ENS Subgraph with no rainbow tables as of
26 May, 2026.\nResults exclude unnormalized names that are formatted as encoded
labelhashes in the ENS Subgraph but are actually known.`}) so the newline is
preserved and will work with whitespace-pre-wrap.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: 989152d1-7aa7-4915-9ce9-208638e2f9b0
📒 Files selected for processing (2)
docs/ensrainbow.io/src/components/molecules/BarChart.tsxdocs/ensrainbow.io/src/components/organisms/FullRainbow.tsx
Y3drk
changed the title
FullRainbow component values and footnote for clarity
Lite PR
Summary
Get the full rainbowsection on ensrainbow.ioWhy
Testing
Notes for Reviewer (Optional)
Pre-Review Checklist (Blocking)