chore(docs): Refine icon wall on the ensnode.io landing page#2217
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughFlatten ENSEcosystem members into a single array and standardize ENSv2 display names; replace ENSv2 Explorer/App SVG path geometry; adjust header/hero z-indexes and spacing; reduce ScrollHeader IntersectionObserver rootMargin to 30px. ChangesENS Ecosystem Members, Logos, and UI
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 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 docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add 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
Refines the ensnode.io landing page “icon wall” / ENS ecosystem members section by adjusting layout spacing and grouping of ecosystem member icons, including a dedicated “official ENSv2” grouping.
Changes:
- Tweaks hero section padding for better vertical balance on smaller breakpoints.
- Adds a new ecosystem member type to separate “official ENSv2” apps from general apps, and updates corresponding display names.
- Restyles the ENS ecosystem members section into a rounded panel with updated spacing/positioning.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| docs/ensnode.io/src/components/overrides/Hero.astro | Adjusts responsive top padding in the landing hero layout. |
| docs/ensnode.io/src/components/organisms/ENSEcosystemMembers/members-collection.tsx | Introduces an OfficialENSApp grouping and updates member labels / ordering. |
| docs/ensnode.io/src/components/organisms/ENSEcosystemMembers/index.tsx | Updates the members section structure/styling (panel container + repositioning). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![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/ensnode.io/src/components/organisms/ENSEcosystemMembers/index.tsx`:
- Line 8: The Tailwind classes `-top-21` and `lg:-top-26` in the
ENSEcosystemMembers component’s container div (the element with className
starting "max-w-[1216px] ... -top-21 sm:top-0 lg:-top-26") are not in the
spacing scale and thus produce no CSS; replace them with Tailwind arbitrary
inset utilities (e.g., `-top-[21px]` and `lg:-top-[26px]`) or add corresponding
`inset` values to the Tailwind config so the container's top offset takes
effect.
🪄 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: a8f45f21-e8c3-4f97-83f9-7669151e400c
📒 Files selected for processing (3)
docs/ensnode.io/src/components/organisms/ENSEcosystemMembers/index.tsxdocs/ensnode.io/src/components/organisms/ENSEcosystemMembers/members-collection.tsxdocs/ensnode.io/src/components/overrides/Hero.astro
![vercel[bot]](https://avatars.githubusercontent.com/in/8329?s=60&v=4){kind=small}
…on wall positioning
…/chore/refine-ensnodeio-icon-wall
…/chore/refine-ensnodeio-icon-wall
Greptile SummaryThis PR refines the icon wall on the ensnode.io landing page with purely visual/UX changes: updated SVG logos, a flattened ecosystem-member data structure, hero section layout improvements, and z-index layering fixes.
Confidence Score: 5/5Safe to merge — all changes are purely visual and confined to the landing page UI. Every change is a UI/UX refinement: SVG logo path updates, a data-structure simplification that has no call-site impact (the removed types were confirmed unused across the codebase), responsive height/overflow adjustments in the hero, and z-index increments that follow a clear intentional layering order. No data mutations, API calls, or shared logic are touched. No files require special attention. Important Files Changed
Reviews (3): Last reviewed commit: "Apply 05/29/26 Slack review requests" | Re-trigger Greptile |
lightwalker-eth
left a comment
lightwalker-eth
left a comment
There was a problem hiding this comment.
@Y3drk Nice updates, thank you! 🫡
Lite PR & rarr; Refine icon wall on the ensnode.io landing page
Summary
icon wallelement on the ensnode.io LP (listed in a related Google Doc)Why
Testing
lintcommand locally to ensure that the migration didn't break anything, and later confirmed that in our CI workflowNotes for Reviewer (Optional)
Pre-Review Checklist (Blocking)