Guide icon indicating copy to clipboard operation
Guide copied to clipboard

Footer icon improvements

Open vivekgh0sh opened this issue 7 months ago • 3 comments

Made some improvements in footer icons as follows:

  • discord - fixed the hover over the discord icon issue where it doesn't change color on hover.
  • github, nostr, youtube, bitcoin tv - fixed the default grey icon and resized to match other logos
  • x - changed the twitter logo to x logo
  • bitcoin tv - made the bitcoin tv svg to match the current logo

Here's a quick comparison: Screenshot from 2025-07-20 18-14-50 Screenshot from 2025-07-20 18-14-42

Docker: removed the version from docker-compose.yml as compose file versions have been deprecated and included a --watch flag to live update changes.

vivekgh0sh avatar Jul 20 '25 13:07 vivekgh0sh

Deploy Preview for bitcoin-design-site ready!

Name Link
Latest commit 45a3e90dc9fb7808883191cd5db409c458d1e634
Latest deploy log https://app.netlify.com/projects/bitcoin-design-site/deploys/687cf379e916f80008c57aab
Deploy Preview https://deploy-preview-1186--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.

netlify[bot] avatar Jul 20 '25 13:07 netlify[bot]

Thanks for the detailed feedback. I should've discussed first before making these changes.

  1. Yes I noticed the links you've visited being greyed out function now, I got confused with the discord logo and twitter logo because in the local build those link checker function were not working.
  2. I am thinking of resizing the discord and X logo to match the current scale of other logos.
  3. It's X - the everything app now.
  4. Agree with the BitcoinTV logo being the misfit, even now it doesn't look aligned.
  5. Agreed
  6. Yes sorry I should have asked before sending the compose changes.

Should I go ahead and make changes to the Discord and X logos?

vivekgh0sh avatar Jul 21 '25 11:07 vivekgh0sh

Sorry for the late reply. Please go for it. Just to summarize again (if I understand correctly):

  • Replace the Twitter bird with the X X
  • Remove the BitcoinTV link (and SVG) altogether, since we don't post there anymore
  • Fixing the Discord SVG

Thanks for contributing.

GBKS avatar Sep 30 '25 12:09 GBKS