feat: add nodejs noodle#2778
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:
📝 WalkthroughWalkthroughReplaces the Press noodle with a Node.js noodle: adds ChangesNoodle logo replacement
Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 4✅ 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 |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
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 `@app/components/Noodle/Nodejs/Logo.vue`:
- Around line 9-12: The external anchor element in Logo.vue that sets
target="_blank" for the Node.js release link (the <a ...
href="https://github.com/nodejs/node-v0.x-archive/releases/tag/v0.0.1"
target="_blank"> anchor) should include rel="noopener noreferrer" to prevent
reverse-tabnabbing; update that anchor element to add rel="noopener noreferrer"
alongside the existing attributes (keeping class="underline" and href intact).
🪄 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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 085ae28a-6d74-4507-ab71-880d1ce386fe
📒 Files selected for processing (4)
app/components/Noodle/Nodejs/Logo.vueapp/components/Noodle/Press/Logo.vueapp/components/Noodle/index.tstest/nuxt/a11y.spec.ts
💤 Files with no reviewable changes (1)
- app/components/Noodle/Press/Logo.vue
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 `@app/components/Tooltip/App.vue`:
- Line 43: Currently attrs is built by spreading consumer props.tooltipAttr
after setting role and id, allowing consumers to override component-owned
accessibility attributes (role and id) and break aria-describedby linkage;
change construction so component-owned attributes (role: 'tooltip' and id:
tooltipId) are applied last or otherwise take precedence over props.tooltipAttr
(e.g., merge as {...props.tooltipAttr, role: 'tooltip', id: tooltipId} or assign
role/id after spreading) so that attrs, tooltipId, and role remain authoritative
and cannot be overridden by props.tooltipAttr.
🪄 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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: b55f916c-ad4c-4b67-a8a9-f14a3a593804
📒 Files selected for processing (2)
app/components/Noodle/Nodejs/Logo.vueapp/components/Tooltip/App.vue
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 `@app/components/Noodle/index.ts`:
- Around line 33-34: The noodle currently sets both date and dateTo to
'2026-05-27', which makes the active-range check in Landing/IntroHeader.vue only
pass at midnight; for a single-day noodle remove the dateTo property and leave
only date so the timezone-aware same-day branch is used. Edit the Noodle export
(the object containing date and dateTo in app/components/Noodle/index.ts) to
omit the dateTo field for this entry (keep date: '2026-05-27') so the
component's single-day logic is triggered.
🪄 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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 5f1ed4b5-ed62-4e81-bcba-be2e72e29008
📒 Files selected for processing (1)
app/components/Noodle/index.ts
🧭 Context
New noodle 🍜