feat(frontend): add offerwall card to the main page#572
Conversation
Deployment results
Logs #23146194859 |
sidvishnoi
left a comment
sidvishnoi
left a comment
There was a problem hiding this comment.
Ask @RabebOthmani to confirm she's ok having this listed on prod on Monday. If not, merge this after Monday release.
will also add the card for wordpress in preparation for release |
sidvishnoi
changed the title
|
@RabebOthmani asked not to add in Monday release. |
|
@DarianM clicking on the Offerwall card takes me to a specific section of the Offerwall page which is the customization section instead of the top of the page. Please fix |
frontend/app/routes/_index.tsx
Outdated
|
|
||
| <p className="text-style-h5 !text-landing-content text-left w-full max-xl:!text-style-small-standard max-xl:text-center"> | ||
| Use integration tools to enable Web Monetization on your site | ||
| through supported platforms like Google Offerwall. |
There was a problem hiding this comment.
Nit: I'm not sure that Offerwall is a platform. Maybe it is. Perhaps change this to "...through supported platforms like Google and WordPress."
|
@DarianM the text on How it works section is split on 2 lines. It looks weird and it is not consistent with the text on the same section on other pages |
|
Not all Font families seem to be working. Arial & Titillium are the same , Inherit and Open Sans also @DarianM @sidvishnoi |
|
In the customization section, When switching between wallets : If I was on preset 3 and I change wallet, I'm staying on preset 3 . it is a different config, it should go to default. Also I noticed that in this case switching between presets, the appearance tab is not expanded @DarianM @sidvishnoi |
|
The done button should be removed for now as it has no function @DarianM @sidvishnoi |
|
@DarianM @sidvishnoi @melissahenderson the link 'View Documentation' on the script pop-up goes to https://webmonetization.org/publishers/offerwall-tool/ I'm not sure if this page exists yet, if not we need to work with the tech writers team to ensure it's ready for release or link removed but we can't have a 404 |
DarianM
changed the title
4 participants
Closes #522