-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
landing page improvements #42
Comments
Sounds good! We can add the official logo to the page right away. Can be on top of the text |
I prefer the logo on top of the text as it represents the logo. @angelofallars |
How about this one? |
Perfect! ✨ |
Add your changes to the PR #45 @thomasbnt |
I leave this PR to @angelofallars if you want 😉 |
The logo shadow was already in #43 |
The logo on top is nice on mobile but at the left on desktop seems clear and better. What did you say @warengonzaga ? We leave that (on top of the text on mobile and desktop) but I remove the shadow? Or I remove shadow + show the logo at the left on desktop and on top on mobile ? |
I guess we can tweak it in the next update. |
Improved and added a description to the landing page at bf64902 |
Users are suggesting adding some information about buymeacoffee.js usage and why they should use this API wrapper to their project and place that information on the front page.
In addition, I guess we can add the official icon to the landing page @thomasbnt? Aside from the text
buymeacoffee.js
The text was updated successfully, but these errors were encountered: