Skip to content

Hide Next button when only one banner theme is active#1232

Open
matthewrball wants to merge 1 commit into
BitcoinDesign:masterfrom
matthewrball:fix/homepage-next-button
Open

Hide Next button when only one banner theme is active#1232
matthewrball wants to merge 1 commit into
BitcoinDesign:masterfrom
matthewrball:fix/homepage-next-button

Conversation

@matthewrball
Copy link
Copy Markdown

Summary

  • The homepage banner theme carousel (js/themes.js) currently has all but one theme commented out for Impact Initiatives 2026
  • With a single active theme, the Next button cycles through a one-element array — clicking it does nothing
  • This hides the Next button when themes.length <= 1, while keeping the Submit your own link visible
  • When themes are re-enabled, the Next button automatically reappears

Test plan

  • Load the homepage — verify "Submit your own" is visible and "Next" is hidden
  • Uncomment one additional theme in js/themes.js — verify both "Submit your own" and "Next" appear and Next cycles themes

@netlify
Copy link
Copy Markdown

netlify Bot commented May 21, 2026

Deploy Preview for bitcoin-design-site ready!

Name Link
🔨 Latest commit 7e2f0f2
🔍 Latest deploy log https://app.netlify.com/projects/bitcoin-design-site/deploys/6a0e5f756b926100092c82f6
😎 Deploy Preview https://deploy-preview-1232--bitcoin-design-site.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 project configuration.

The themes array currently has all but one theme disabled for
Impact Initiatives 2026. With a single active theme, the Next
button does nothing — hide it while keeping Submit your own visible.
@matthewrball matthewrball force-pushed the fix/homepage-next-button branch from e12ced8 to 7e2f0f2 Compare May 21, 2026 01:27
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