Skip to content

Updating Navbar#543

Closed
prayagd wants to merge 4 commits into
stagingfrom
navbar-update
Closed

Updating Navbar#543
prayagd wants to merge 4 commits into
stagingfrom
navbar-update

Conversation

@prayagd
Copy link
Copy Markdown
Member

@prayagd prayagd commented Apr 2, 2025

  • Remove Sell crypto, How it works and community
  • Add Support, Docs and FAQ
  • Links also updated

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 2, 2025

Deploy Preview for pendulum-pay ready!

Name Link
🔨 Latest commit 9efed52
🔍 Latest deploy log https://app.netlify.com/sites/pendulum-pay/deploys/67ed0db0063b5f0008f20f6c
😎 Deploy Preview https://deploy-preview-543--pendulum-pay.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.

prayagd added 3 commits April 2, 2025 15:28
Updated the links to the new navbar items
{ title: t('components.navbar.community'), href: 'https://www.vortexfinance.co/#call-to-action' },
{ title: t('components.navbar.support'), href: 'https://pendulum.gitbook.io/vortex/support-and-contact' },
{ title: t('components.navbar.docs'), href: 'https://pendulum.gitbook.io/vortex' },
{ title: t('components.navbar.faq'), href: 'https://app.vortexfinance.co/#faq' },
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What's the expected behavior for clicking on the FAQ link? Do you want it to scroll down on the same page or open a new page?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

scroll down to the same page to the faq section

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This doesn't work in that case then. For one, you are always putting the production address so on deploy previews etc. it will always open a new page. If you feel like it, feel free to try troubleshooting it yourself with AI. Otherwise one of the @pendulum-chain/devs needs to add that before we can merge.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ok let me give a try

@ebma ebma closed this Apr 30, 2025
@ebma
Copy link
Copy Markdown
Member

ebma commented Apr 30, 2025

Superseded by #605.

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.

2 participants