-
Notifications
You must be signed in to change notification settings - Fork 81
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
feat: implement new website homepage #1949
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/contentful-apps/forma-36/8zvg9eHmS851BjbUiDhwPAK34T4N |
|
@domarku feedback from my side is that I would not expect those cards are links. I think we should have CTA in each card if we would like to be clear about it. Or maybe it's just me? |
My 2 cents on this that it could use some colors, maybe icons on cards or something. It kinda look cheap 🥲 |
Alright friends, thanks for the feedback! What I'm getting is "back to the drawing board". It's not yet ready for a PR. Let me iterate on it a bit and get more of your feedback on the design before we put it up :) |
e266abe
to
3788810
Compare
@gui-santos let's wait for Monday to merge the homepage. I need to prepare the UI Kit for publishing today. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
5b35e2f
to
a70866e
Compare
a70866e
to
bde1d2f
Compare
Purpose of PR
PR Checklist
readme.md
file(s)