Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chrome extension open to new page #296

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Adnan0061
Copy link

@Adnan0061 Adnan0061 commented Sep 21, 2024

Onboarding Page Update

  • Purpose:
    Update the onboarding page to improve the user experience.
  • Key Changes:
    • Added a target="_blank" attribute to the link for downloading the Chrome extension, to open it in a new tab.
    • Removed unnecessary whitespace.
  • Impact:
    The changes will provide a better user experience by opening the extension download link in a new tab, keeping the user on the onboarding page.

✨ Generated with love by Kaizen ❤️

Original Description

while onboarding if click to chrome extension it movers out of onboarding flow that's why it needs to open in a new tab
Copy link
Contributor

kaizen-bot bot commented Sep 21, 2024

🔍 Code Review Summary

All Clear: This commit looks good! 👍

Overview

  • Total Feedbacks: 0 (Critical: 0, Refinements: 0)
  • Files Affected: 0
  • Code Quality: [█████████████████░░░] 85% (Good)

✨ Generated with love by Kaizen ❤️

Useful Commands
  • Feedback: Reply with !feedback [your message]
  • Ask PR: Reply with !ask-pr [your question]
  • Review: Reply with !review
  • Explain: Reply with !explain [issue number] for more details on a specific issue
  • Ignore: Reply with !ignore [issue number] to mark an issue as false positive
  • Update Tests: Reply with !unittest to create a PR with test changes

@@ -147,7 +147,7 @@ function StepOne({ currStep }: { currStep: number }) {
<ol className="text-lg space-y-3">
<li>
Download the chrome extension{" "}
<a className="underline underline-offset-2" href="/extension">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment: Missing rel attribute for external links

Solution: Add rel='noopener noreferrer' to the anchor tag.

Suggested change
<a className="underline underline-offset-2" href="/extension">
<a className="underline underline-offset-2" href="/extension" target="_blank" rel="noopener noreferrer">

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant