shapeshift / web

ShapeShift Web
https://app.shapeshift.com
MIT License
159 stars 180 forks source link

FOX Farm CTA #1550

Closed SS-FOX-McCloud closed 2 years ago

SS-FOX-McCloud commented 2 years ago

Overview

Overview As a user I want to be aware and be routed to the FOX farm page and all associated opportunities via the DeFi earn tab (https://app.shapeshift.com/defi/earn) on the web app. This should be done via a CTA (call to action) banner.

COPY: "Earn 60%+ APR* by farming your ETH-FOX liquidity pool tokens"

There should also be a button that says "Start Farming FOX" which will link to https://fox.shapeshift.com/fox-farming

This banner/button should live at the top of the earn page. Please see the attached image for additional details/design.

References and additional details

Web. Need additional detail from engineering.

Acceptance Criteria

A CTA banner will live at the top of the earn section with a clickable button linking to https://fox.shapeshift.com/fox-farming

Need By Date

n/a

Screenshots/Mockups

[No response](https://www.sketch.com/s/5dcee031-1213-44a1-a94e-e3439cfb4d17/a/v8nnO9J

image)

Estimated effort

2 hours

0xdef1cafe commented 2 years ago

@SS-FOX-McCloud what does this look like on tablet/mobile?

0xdef1cafe commented 2 years ago

nothing required here on an engineering basis, once you can clarify responsive views this can be bountied.

SS-FOX-McCloud commented 2 years ago

Hoff working on mobile mockup...will mark ready for bounty once completed.

SS-FOX-McCloud commented 2 years ago

Here is the CTA on mobile. This should be ready for bounty now. Please let me know if there are any other questions!

MobileCTA

brymut commented 2 years ago

Where would the button go for the Button in the CTA on mobile? @SS-FOX-McCloud

0xdef1cafe commented 2 years ago

looks great @brymut but will await @SS-FOX-McCloud input regarding the button

SS-FOX-McCloud commented 2 years ago

https://fox.shapeshift.com/fox-farming

@brymut The button should still go to the FOX farm page: https://fox.shapeshift.com/fox-farming

Is that what you are asking?

brymut commented 2 years ago

@brymut The button should still go to the FOX farm page: https://fox.shapeshift.com/fox-farming

Is that what you are asking?

There is no CTA button in the mobile mock-up you provided.

SS-FOX-McCloud commented 2 years ago

@brymut ah, gotcha! Could the entire banner/CTA be "clickable" on mobile and direct the user to https://fox.shapeshift.com/fox-farming ?

NeOMakinG commented 2 years ago

@brymut I would suggest to remove the href from the button but keep it like it is, and apply the href to the whole container even on desktop, so the whole banner is clickable on desktop too

wdyt @SS-FOX-McCloud ?

SS-FOX-McCloud commented 2 years ago

@brymut I would suggest to remove the href from the button but keep it like it is, and apply the href to the whole container even on desktop, so the whole banner is clickable on desktop too

wdyt @SS-FOX-McCloud ?

Sure!- That would work great! good idea!

NeOMakinG commented 2 years ago

@brymut ah, gotcha! Could the entire banner/CTA be "clickable" on mobile and direct the user to https://fox.shapeshift.com/fox-farming ?

Not sure this would work as there wouldn't be any actual CTA maybe look into the screenshots in the PR and provide feedback on those?

Damn, I still agree with you, having a CTA is a good practice, we shouldn't remove it on mobile, or the user doesn't know that it's clickable :p

SS-FOX-McCloud commented 2 years ago

@brymut ah, gotcha! Could the entire banner/CTA be "clickable" on mobile and direct the user to https://fox.shapeshift.com/fox-farming ?

Not sure this would work as there wouldn't be any actual CTA maybe look into the screenshots in the PR and provide feedback on those?

Damn, I still agree with you, having a CTA is a good practice, we shouldn't remove it on mobile, or the user doesn't know that it's clickable :p

Thanks for the discussion. On mobile since there isn't room for the button (and most users will access FOX farm from a desktop anyways, I assume with the wallet connectivity being better on desktop) Can we just update the copy on mobile only to "Click here to Earn 60%+ APR* by farming your ETH-FOX liquidity pool tokens" and then make the entire banner linkable as previously discussed? Let me know if you dont think this is doable. Thanks!

NeOMakinG commented 2 years ago

@brymut ah, gotcha! Could the entire banner/CTA be "clickable" on mobile and direct the user to https://fox.shapeshift.com/fox-farming ?

Not sure this would work as there wouldn't be any actual CTA maybe look into the screenshots in the PR and provide feedback on those?

Damn, I still agree with you, having a CTA is a good practice, we shouldn't remove it on mobile, or the user doesn't know that it's clickable :p

Thanks for the discussion. On mobile since there isn't room for the button (and most users will access FOX farm from a desktop anyways, I assume with the wallet connectivity being better on desktop) Can we just update the copy on mobile only to "Click here to Earn 60%+ APR* by farming your ETH-FOX liquidity pool tokens" and then make the entire banner linkable as previously discussed? Let me know if you dont think this is doable. Thanks!

Should be doable and easier if we make the banner fully clickable even on desktop, thanks for your time and feedbacks :+1:

SS-FOX-McCloud commented 2 years ago

@brymut ah, gotcha! Could the entire banner/CTA be "clickable" on mobile and direct the user to https://fox.shapeshift.com/fox-farming ?

Not sure this would work as there wouldn't be any actual CTA maybe look into the screenshots in the PR and provide feedback on those?

Damn, I still agree with you, having a CTA is a good practice, we shouldn't remove it on mobile, or the user doesn't know that it's clickable :p

Thanks for the discussion. On mobile since there isn't room for the button (and most users will access FOX farm from a desktop anyways, I assume with the wallet connectivity being better on desktop) Can we just update the copy on mobile only to "Click here to Earn 60%+ APR* by farming your ETH-FOX liquidity pool tokens" and then make the entire banner linkable as previously discussed? Let me know if you dont think this is doable. Thanks!

Should be doable and easier if we make the banner fully clickable even on desktop, thanks for your time and feedbacks 👍

Ok- Lets scrap the button on both desktop and mobile, add the "Click here..." updated copy to mobile and Desktop, and make the banner clickable on both as well. Thanks!

gomesalexandre commented 2 years ago

@SS-FOX-McCloud Can we update the issue with the new spec to make it easier for reviewers? 🙏🏽