Guide
Guide copied to clipboard
Footer icon improvements
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:
Docker: removed the version from docker-compose.yml as compose file versions have been deprecated and included a --watch flag to live update changes.
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...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify project configuration.
Thanks for the detailed feedback. I should've discussed first before making these changes.
- 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.
- I am thinking of resizing the discord and X logo to match the current scale of other logos.
- It's X - the everything app now.
- Agree with the BitcoinTV logo being the misfit, even now it doesn't look aligned.
- Agreed
- Yes sorry I should have asked before sending the compose changes.
Should I go ahead and make changes to the Discord and X logos?
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.