fix(sponsorship): open sponsor links in a new tab

[Renanhcorrea]

✨ Codu Pull Request 💻

Fixes #1297

Pull Request details

Fixed inconsistent behavior on the /sponsorship page where only one sponsor link opened in a new tab.

Any Breaking changes

None

Associated Screenshots

None

[Optional] What gif best describes this PR or how it makes you feel

None

[vercel]

@Renanhcorrea is attempting to deploy a commit to the Codú Team on Vercel.

A member of the Team first needs to authorize it.

[github-actions]

Hello @Renanhcorrea, thanks for opening your first Pull Request. The maintainers will review this Pull Request and provide feedback as soon as possible. Keep up the great work!

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

Sponsor images on the sponsorship page were made clickable: each <img> is now wrapped with a Link pointing to the sponsor's external URL and includes target="_blank" and rel="noopener noreferrer".

Changes

Cohort / File(s)	Summary
Sponsor links enhancement app/(app)/sponsorship/page.tsx	Replaced static sponsor <img> elements by wrapping them in Link components with href to sponsor pages; added target="_blank" and rel="noopener noreferrer" while preserving existing image attributes (src, alt, sizing).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• fix(sponsorship): open sponsor links in a new tab #1307 — Modifies the same app/(app)/sponsorship/page.tsx to wrap sponsor images with external Link components and add target="_blank"/rel="noopener noreferrer".

Poem

🐰
Click the logos, off they hop,
New tabs open—what a pop!
I nibble bytes and cheer the view,
Sponsors linked, the page anew. 🥕✨

Pre-merge checks and finishing touches

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The pull request title "fix(sponsorship): open sponsor links in a new tab" clearly and specifically describes the main change in the changeset. The raw summary confirms that the primary modification wraps sponsor images in external Link components, making them clickable with target="_blank" attributes, which directly corresponds to the title's intent. The title is concise, uses conventional commit formatting with a "fix" scope, and accurately conveys the purpose of the changes without ambiguity or vague language.
Linked Issues Check	✅ Passed	The linked issue #1297 requires adding missing sponsor links on the sponsorship page to make all sponsor logos consistently clickable. The code changes directly address this objective by wrapping multiple sponsor images in Link components with href attributes, replacing static images with clickable links. The implementation also includes appropriate security attributes (target="_blank" with rel="noopener noreferrer") for external links. The changes fully satisfy the primary coding requirement specified in the linked issue.
Out of Scope Changes Check	✅ Passed	All changes in this pull request are directly scoped to addressing the objective in linked issue #1297. The modifications are limited to wrapping sponsor images in Link components within app/(app)/sponsorship/page.tsx, which is the precise scope of the issue. No other files are modified, no component signatures are altered, and no unrelated functionality has been introduced. The addition of target="_blank" and rel="noopener noreferrer" attributes is a standard best practice for external links and is appropriately aligned with the PR's intent to open sponsor links in new tabs.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between aac418a and 26c3b98.

📒 Files selected for processing (1)

• app/(app)/sponsorship/page.tsx (1 hunks)

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (1)

app/(app)/sponsorship/page.tsx (1)

101-101: Standardize attribute order for consistency.

The attributes are in a different order (rel, href, target) compared to the other sponsor links which use (href, target, rel). While functionally equivalent, consistent ordering improves code maintainability.

Apply this diff to match the attribute order used in other sponsor links:

-                <Link rel="noopener noreferrer" href="https://www.wework.com/" target="_blank">
+                <Link href="https://www.wework.com/" target="_blank" rel="noopener noreferrer">

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d810785 and aac418a.

⛔ Files ignored due to path filters (1)

• package-lock.json is excluded by !**/package-lock.json, !**/*.json

📒 Files selected for processing (1)

• app/(app)/sponsorship/page.tsx (1 hunks)

🧰 Additional context used

🧬 Code graph analysis (1)

app/(app)/sponsorship/page.tsx (2)

components/ui-components/link.tsx (1)

• Link (12-22)

components/PromptService/PromptLink.tsx (2)

• LinkProps (22-86)
• LinkProps (9-19)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: e2e

🔇 Additional comments (3)

app/(app)/sponsorship/page.tsx (3)

83-89: LGTM!

The external link implementation is correct with proper security attributes and consistent formatting.

---

92-98: LGTM!

The external link implementation is correct with proper security attributes and consistent formatting.

---

74-107: External sponsor links implemented correctly.

All external sponsor links now consistently open in new tabs with proper security attributes (target="_blank" and rel="noopener noreferrer"), which addresses the issue mentioned in #1297.

Note: The NineDots sponsor link (lines 65-71) uses internal navigation (/company/ninedots) and does not open in a new tab, which follows standard UX practices for internal links. If all sponsor links should behave identically regardless of destination, you may want to add target="_blank" to the NineDots link as well.

[NiallJoeMaher]

🌮

[NiallJoeMaher]

🌮