Skip to content

bworg: use Tivra's site for bisonwallet.org #3222

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

buck54321
Copy link
Member

Tivra made this great little one-page site. I'm putting it up at bisonwallet.org for now. I'm opening the PR to give a place for people to recommend alterations.

Copy link
Member

@jzbz jzbz left a comment

Choose a reason for hiding this comment

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

Need to change the two links pointing to https://github.com/decred/bisonrelay to https://github.com/decred/dcrdex/.

@jzbz
Copy link
Member

jzbz commented Mar 31, 2025

@Tsunekazu It's up.
Making a few notes:

  • Font next to logo at in the header has some rendering issues.
  • All fonts should be hosted locally, eliminate all Google links (gstatic, googleapis).
  • Support link should either link to Decred support page or directly to a support channel.

Copy link
Member

@JoeGruffins JoeGruffins left a comment

Choose a reason for hiding this comment

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

BisonWallet is cut off in the top left. The coin icons are cut off in odd places and otherwise don't look great. But awesome he was able to make this using ai.
image

Copy link
Contributor

@dev-warrior777 dev-warrior777 left a comment

Choose a reason for hiding this comment

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

image

If already seen in DCR DEX channel, apologies .. minor issue I think

@martonp
Copy link
Collaborator

martonp commented Apr 1, 2025

Uploading Screenshot 2025-04-01 at 9.31.37 AM.png…

POLY should be POL

@buck54321
Copy link
Member Author

I think I've addressed everything except for @martonp's POLY find. I've got that done locally and I'll push it soon.

@jzbz
Copy link
Member

jzbz commented Apr 2, 2025

One other small issue the life preserver icon in the support section isn't properly aligned like all the others.

image

@buck54321
Copy link
Member Author

One other small issue the life preserver icon in the support section isn't properly aligned like all the others.

Yeah, I messed that up. I have it fixed locally. I'm thinking about removing those icons through.

@JoeGruffins
Copy link
Member

Coin Icons look good now.

@JoeGruffins
Copy link
Member

LGTM

@ukane-philemon
Copy link
Contributor

  1. The Get Bison wallet section hides the navbar during scrolls.
Screenshot 2025-05-18 at 8 11 55 PM
  1. I suggest the menu dropdown should close if the user clicks outside of the menu dropdown.

  2. Add Brand Resources page/link to bisonwallet.org #3279 (This can be handled as a follow up)

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.

6 participants