chore(docs): New batch of UI refinements to ensnode.io docs #2221
Conversation
…color inside <Aside> component
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
Warning Review limit reached
More reviews will be available in 47 minutes and 15 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughAdds a new ChangesDocumentation UI styling refinement
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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 |
![vercel[bot]](https://avatars.githubusercontent.com/in/8329?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
This PR refines styling in the ensnode.io docs site, primarily adjusting link/button hover behaviors and minor layout details in Omnigraph static example components and hosted instance UI.
Changes:
- Prevent
<Aside/>links from changing to blue on hover by explicitly setting the hover color instarlight.css. - Update Omnigraph static example hint/action-bar link/button styling (underline offset + hover background behavior).
- Minor hosted instance UI tweaks (transition type and preventing “Connect with ENSAdmin” from wrapping).
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| docs/ensnode.io/src/styles/starlight.css | Keeps aside link hover color consistent with the aside theme color. |
| docs/ensnode.io/src/components/molecules/omnigraph-static-example/StaticExamplePlaygroundHint.astro | Updates hosted-instance link styling (underline/offset/transition utilities). |
| docs/ensnode.io/src/components/molecules/omnigraph-static-example/StaticExampleActionBar.astro | Adjusts action button hover behavior to use background color transition. |
| docs/ensnode.io/src/components/molecules/HostedEnsNodeInstance.astro | Changes button transition and adds a no-wrap utility to the ENSAdmin label. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Greptile SummaryThis PR extracts the repeated ENSAdmin CTA button markup into a shared
Confidence Score: 5/5Pure UI/CSS refactor with no logic changes; safe to merge. All changes are scoped to docs UI components and styles. The button extraction is clean, the CSS bug fixes are correct, and the previously-flagged Tailwind transition class is properly resolved. The only notable deviation is the non-standard Astro prop interface in the new component, which is a style concern with no runtime impact. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[ENSAdminCTAButton.astro\nnew shared atom] --> B[HostedEnsNodeInstance.astro\nConnects with ENSAdmin]
A --> C[StaticExampleActionBar.astro\nRuns in ENSAdmin]
C -->|adminUrl guard| D{adminUrl?}
D -- yes --> A
D -- no --> E[render nothing]
B --> F[not-content wrapper]
F --> A
Reviews (7): Last reviewed commit: "Apply AI agents' suggestions, pt.2" | Re-trigger Greptile |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?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/ensnode.io/src/components/molecules/omnigraph-static-example/StaticExamplePlaygroundHint.astro`:
- Line 13: The class string in StaticExamplePlaygroundHint.astro contains a
non-standard utility "transition-underline" which likely does nothing; update
the class list in the component (the string containing "text-sm font-semibold
... transition-underline duration-200") to use a valid Tailwind transition
utility such as "transition-[text-underline-offset] duration-200" or
"transition-all duration-200" so the underline-offset changes animate correctly.
🪄 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: c1c74957-1c2f-4287-9b9e-3ec920c8dfa1
📒 Files selected for processing (4)
docs/ensnode.io/src/components/molecules/HostedEnsNodeInstance.astrodocs/ensnode.io/src/components/molecules/omnigraph-static-example/StaticExampleActionBar.astrodocs/ensnode.io/src/components/molecules/omnigraph-static-example/StaticExamplePlaygroundHint.astrodocs/ensnode.io/src/styles/starlight.css
Y3drk
changed the title
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 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/atoms/ENSAdminCTAButton.astro`:
- Around line 16-23: ENSAdminCTAButton currently accepts ensAdminHref?: string
but can render an <a href={undefined}>; either make ensAdminHref required on
ENSAdminCTAButton's props (update callers like StaticExampleActionBar,
resolveOmnigraphStaticExample, and HostedEnsNodeInstance to always pass the
string) or guard inside ENSAdminCTAButton: if ensAdminHref is falsy render a
non-anchor fallback (e.g., a <button> or <span> with the same styling,
aria-disabled and no href) to avoid broken links and improve accessibility;
update ENSAdminCTAButton's prop typing and the render branch referencing
ensAdminHref accordingly.
In
`@docs/ensnode.io/src/components/molecules/omnigraph-static-example/StaticExampleActionBar.astro`:
- Around line 22-23: actionButtonClass duplicates the styles string used as
styles in ENSAdminCTAButton.astro; extract the shared class into a single
exported constant (e.g. CTA_BUTTON_CLASS) in a small helper module and import
that constant into both StaticExampleActionBar.astro and
ENSAdminCTAButton.astro, replacing the local actionButtonClass and styles
definitions so both components consume the same value and avoid visual drift.
🪄 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: 3a043e37-29d2-484d-a0e4-6d11b41808b4
📒 Files selected for processing (5)
docs/ensnode.io/src/components/atoms/ENSAdminCTAButton.astrodocs/ensnode.io/src/components/molecules/HostedEnsNodeInstance.astrodocs/ensnode.io/src/components/molecules/omnigraph-static-example/StaticExampleActionBar.astrodocs/ensnode.io/src/components/molecules/omnigraph-static-example/StaticExamplePlaygroundHint.astrodocs/ensnode.io/src/styles/starlight.css
💤 Files with no reviewable changes (1)
- docs/ensnode.io/src/styles/starlight.css
lightwalker-eth
left a comment
lightwalker-eth
left a comment
There was a problem hiding this comment.
@Y3drk Looks good, thanks 👍
Lite PR → New batch of refinements to ensnode.io docs
Summary
ENSAdmin CTAbutton to a separate component to avoid code duplication<Aside/>components, where the links would change the color to blue when hovered.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)