Skip to content
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

Add /faucet page #158

Merged
merged 1 commit into from
Feb 13, 2023
Merged

Add /faucet page #158

merged 1 commit into from
Feb 13, 2023

Conversation

DoubleOTheven
Copy link
Contributor

@DoubleOTheven DoubleOTheven commented Feb 11, 2023

Code changes for this Git issue

  • Add /faucet path with prettified faucet
  • Update /testnet overview docs to link to the new faucet page
  • Responsive layouts for mobile, tablet, and desktop
  • Add Montserrat font for /faucet page

The React component for Recaptcha is not respecting the theme prop so the color is not turning to black as specified in the illustration here. I personally think it looks good with the clean white, but can change this if it is a deal breaker.

Desktop

Screen Shot 2023-02-10 at 7 33 10 PM

Screen Shot 2023-02-10 at 7 35 16 PM

iPhone 11

Screen Shot 2023-02-10 at 7 33 54 PM

Screen Shot 2023-02-10 at 7 34 56 PM

Updated Testnet Docs

Screen Shot 2023-02-10 at 7 37 59 PM

--ifm-color-primary-light: #bd83fb;
--ifm-color-primary-lighter: #bd83fb;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Accidental remove?

@cmichi
Copy link
Collaborator

cmichi commented Feb 11, 2023

Just as an FYI: This comment explains how to have the faucet work despite localhost.

@cmichi
Copy link
Collaborator

cmichi commented Feb 11, 2023

Thank you, Sam! The color is fine.

You can follow the above linked comment to test it locally, you'll then see that the hash label is too big for the box:

screenshot-dev use ink_8080-2023 02 11-02_20_08

Could you display something like this instead:

Funds have been sent!

Click here to view the transaction.

@cmichi cmichi merged commit e2fad4a into master Feb 13, 2023
@cmichi cmichi deleted the sr/prettify-faucet branch February 13, 2023 07:09
@cmichi
Copy link
Collaborator

cmichi commented Feb 13, 2023

@DoubleOTheven I've merged, but please still fix the above things.

I've noticed that the font doesn't seem to load correctly (only tested in on Safari):

Bildschirm­foto 2023-02-13 um 12 25 04

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.

3 participants