Skip to content

add reusable ExtensionLinks component, use on /install page#510

Merged
sidvishnoi merged 5 commits intomainfrom
extension-icons-component
Mar 25, 2025
Merged

add reusable ExtensionLinks component, use on /install page#510
sidvishnoi merged 5 commits intomainfrom
extension-icons-component

Conversation

@sidvishnoi
Copy link
Collaborator

@sidvishnoi sidvishnoi commented Mar 19, 2025

  • Replace the extension links on /install page and /supporters/get-started with a reusable component
    • Minor UI change, but I moved GitHub link into same line as other icons. To keep it simple.
    • Added shadow styles same as /supporters/get-started page.
  • The component has a utm prop, so we can track from where user came to the extension from within website, and as a general marketing campaign.
    • Ideally, we want to pass /install page's UTM params (assuming the page stays) to the component, but not possible with static builds without JS.
  • Replace github-icon.svg with a square one (earlier one as 98x96).

@sidvishnoi sidvishnoi requested a review from JoblersTune March 19, 2025 08:53
@netlify
Copy link

netlify bot commented Mar 19, 2025

Deploy Preview for webmonetization-preview ready!

Name Link
🔨 Latest commit 0535bb9
🔍 Latest deploy log https://app.netlify.com/sites/webmonetization-preview/deploys/67e2a44a28bede0008cc421f
😎 Deploy Preview https://deploy-preview-510--webmonetization-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@w3cbot
Copy link

w3cbot commented Mar 19, 2025

sidvishnoi marked as non substantive for IPR from ash-nazg.

@sidvishnoi sidvishnoi merged commit 9092e66 into main Mar 25, 2025
6 checks passed
@sidvishnoi sidvishnoi deleted the extension-icons-component branch March 25, 2025 13:19
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.

4 participants