Skip to content

docs: open card link in new tab - #76

Merged
calebephrem merged 1 commit into
mainfrom
dev
Aug 30, 2026
Merged

docs: open card link in new tab#76
calebephrem merged 1 commit into
mainfrom
dev

Conversation

@calebephrem

Copy link
Copy Markdown
Member

What does this PR do?

Makes card links open in new tab

Closes

Type of change(s)

  • Bug fix
  • New feature
  • Documentation update
  • Style / UI change
  • Refactor (no functional change)
  • Performance improvement
  • Other

Checklist

  • I've tested this change locally and it works as expected
  • bun run lint or npm run lint completes without warnings or errors
  • bun run build or npm run build completes successfully
  • My commit messages follow Conventional Commits (e.g. feat: ..., fix: ..., chore: ...)
  • I've updated relevant documentation (README, comments, etc.) if needed
  • I've removed any temporary debugging code (e.g. console.log)

@devhub-bot devhub-bot Bot added the docs Improvements or additions to documentation label Aug 30, 2026
@beetle-ai

beetle-ai Bot commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Summary by Beetle

This PR implements a UX improvement for external links by adding target="_blank" and rel="noopener noreferrer" attributes to all card links across three blog post files. The change ensures that when users click on external resource links (GitHub repositories, social media posts, documentation), they open in a new browser tab rather than navigating away from the current page, improving user experience and reducing friction when exploring referenced content.

📁 File Changes Summary (Consolidated across all commits):

File Status Changes Description
content/blog/android-developer-verification.md
content/blog/discord-grid-ban-explained.md
content/blog/top-15-amazing-github-repos.md
Modified +17/-17 Updated all external card links to open in new tabs with target="_blank" rel="noopener noreferrer" attributes. This improves UX by preventing navigation away from blog posts when users click on referenced external resources (GitHub repos, documentation, social media posts). Applied consistently across 17 card link elements.

Total Changes: 3 files changed, +17 additions, -17 deletions

🎯 Key Changes:

  • External Link Behavior: All <a class="card"> elements now include target="_blank" to open links in new tabs
  • Security Enhancement: Added rel="noopener noreferrer" to prevent potential security vulnerabilities (window.opener access) and referrer leakage
  • Consistent Application: Changes applied uniformly across all blog post card links (17 instances total)
  • User Experience: Readers can now explore referenced resources without losing their place in the blog post
  • Scope: Affects blog content files with external resource references (GitHub repositories, Android documentation, social media posts)

📊 Impact Assessment:

  • Security: ✅ Positive - The addition of rel="noopener noreferrer" follows security best practices for external links. This prevents the opened page from accessing the window.opener property and removes the Referer header, protecting user privacy and preventing potential security exploits.
  • Performance: ✅ Neutral - No performance impact. The HTML attribute additions are minimal and have no runtime overhead. Browser tab management is handled natively by the browser.
  • Maintainability: ✅ Positive - Establishes a consistent pattern for external links across the blog. Future blog posts should follow this same pattern, improving code consistency and reducing the likelihood of security oversights.
  • Testing: ✅ Recommended - While this is a straightforward HTML change, consider:
  • Manual testing to verify links open in new tabs across different browsers
  • Accessibility testing to ensure screen readers properly announce the link behavior
  • No automated tests required for this change, but documentation should clarify the link behavior pattern for content contributors
⚙️ Settings

Severity Threshold: Medium — Balanced feedback — medium and high severity issues only.Change in Settings
Custom Rules: Define your own review rules — Set Custom Rules
PR Summary: Configure PR summary — Change in Settings

📖 User Guide
  • Once repos are connected, PR analysis is automatically enabled. You can disable analysis for this repo from beetleai.dev/analysis
  • Comment @beetle on any PR to start analysis manually
  • Comment @beetle stop to stop any ongoing analysis

Follow us: Beetle · X · LinkedIn

@devhub-bot

devhub-bot Bot commented Aug 30, 2026

Copy link
Copy Markdown

Note

Linting checks passed successfully 🎉

All formatting and code quality checks are clean.

You're good to merge 🚀

@beetle-ai

beetle-ai Bot commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

✅ You're good to merge this PR! No issues found. Great job!

Settings
⚙️ Settings

Severity Threshold: Medium — Balanced feedback — medium and high severity issues only.Change in Settings
Custom Rules: Define your own review rules — Set Custom Rules
PR Summary: Configure PR summary — Change in Settings

📖 User Guide
  • Once repos are connected, PR analysis is automatically enabled. You can disable analysis for this repo from beetleai.dev/analysis
  • Comment @beetle on any PR to start analysis manually
  • Comment @beetle stop to stop any ongoing analysis

@calebephrem
calebephrem merged commit 8de6320 into main Aug 30, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make every card open in new tab in blog posts

1 participant