Skip to content

feat: support target prop in Footer links#2568

Merged
hunterbecton merged 1 commit intointegrations/makeswiftfrom
hunter/eng-10346-open-in-new-tab-property-not-working-in-site-footer
Sep 2, 2025
Merged

feat: support target prop in Footer links#2568
hunterbecton merged 1 commit intointegrations/makeswiftfrom
hunter/eng-10346-open-in-new-tab-property-not-working-in-site-footer

Conversation

@hunterbecton
Copy link
Copy Markdown
Contributor

What/Why?

  • adds support for target in Footer links

Testing

CleanShot.2025-09-02.at.11.33.06.mp4

@hunterbecton hunterbecton requested a review from a team September 2, 2025 15:34
@hunterbecton hunterbecton requested a review from a team as a code owner September 2, 2025 15:34
@linear
Copy link
Copy Markdown

linear Bot commented Sep 2, 2025

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Sep 2, 2025

🦋 Changeset detected

Latest commit: 4145846

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@bigcommerce/catalyst-makeswift Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 2, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
catalyst-b2b Ready Ready Preview Comment Sep 2, 2025 4:43pm
catalyst-canary Ready Ready Preview Comment Sep 2, 2025 4:43pm
3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
catalyst Ignored Ignored Sep 2, 2025 4:43pm
catalyst-au Ignored Ignored Sep 2, 2025 4:43pm
catalyst-uk Ignored Ignored Sep 2, 2025 4:43pm

Copy link
Copy Markdown
Contributor

@matthewvolk matthewvolk left a comment

Choose a reason for hiding this comment

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

Looks good, when target is undefined it will simply fall back to not opening a new tab.

@hunterbecton hunterbecton force-pushed the hunter/eng-10346-open-in-new-tab-property-not-working-in-site-footer branch from e066968 to 4145846 Compare September 2, 2025 16:29
Copy link
Copy Markdown
Contributor

@matthewvolk matthewvolk left a comment

Choose a reason for hiding this comment

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

Re-approving after added changeset 👍

@hunterbecton hunterbecton merged commit f15e9db into integrations/makeswift Sep 2, 2025
5 checks passed
@hunterbecton hunterbecton deleted the hunter/eng-10346-open-in-new-tab-property-not-working-in-site-footer branch September 2, 2025 16:40
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.

3 participants