propose favicon, tos and rewards #8
Conversation
| </div> | ||
| </section> | ||
| <section class="section-tos"><div><a href="#"></a>Terms of Service</div></section> |
There was a problem hiding this comment.
TOS is a good idea but let's leave it out until ready instead of leaving a stub.
| <h2>Earnings</h2> | ||
| <h2><strong>What You can Earn</strong></h2> |
There was a problem hiding this comment.
The earnings section is currently disabled because functionality hasn't been implemented yet. I was hoping to tackle that soon.
| @@ -53,12 +53,14 @@ const delegateAddress = `tz1R4PuhxUxBBZhfLJDx2nNjbr7WorAPX1oC` | |||
| background-color: $background-color; | |||
| display: inline-grid; | |||
| grid-template-columns: minmax(0, auto) minmax(0, auto); | |||
| cursor: pointer; | |||
| <section class="section-earnings"> | ||
| <h1> Current APY 5.5% </h1> <!-- This of course can be pulled from where ever. --> | ||
| </section> |
There was a problem hiding this comment.
I prefer to leave this out until the implementation of a function that pulls the value from a node or api so I'll keep this idea as a feature request.
|
|
||
| .section-earnings { | ||
| text-align: center; | ||
| margin-top: 10%; | ||
| } | ||
|
|
||
| h1 { | ||
| color: white; | ||
| font-size: clamp(2rem, 10vw, 4rem); | ||
|
|
||
| img { | ||
| display: inline-block; | ||
| vertical-align: text-bottom; | ||
| height: clamp(3rem, 15vw, 5rem); | ||
| vertical-align: middle; | ||
| } | ||
| } | ||
|
|
There was a problem hiding this comment.
The CSS margins for mobile have disappeared but I'm not quite sure what's the cause.
| { | ||
| rel: 'icon', | ||
| type: 'image/png', | ||
| href: 'btz.png' | ||
| }, |
There was a problem hiding this comment.
We definitely need a favicon but the one provided is difficult to see.
The image size is also quite large. I usually use https://realfavicongenerator.net/ to generate the favicon.
Propose a favicon, and placement of Terms of Service and Rewards.
Also adds a pointer to the copy-address field.