-
Notifications
You must be signed in to change notification settings - Fork 603
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
Update: Add AI Gateway tile to landing page #8123
Conversation
✅ Deploy Preview for kongdocs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
AI gateway text looks good, I left some feedback on the tile #s
@@ -377,7 +377,7 @@ header.navbar { | |||
.cards-container { | |||
display: grid; | |||
grid-gap: 24px; | |||
grid-template-columns: 1fr 1fr 1fr; | |||
grid-template-columns: 1fr 1fr; |
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.
I think I prefer the 3 tiles. Would a row of 4 and a row of 3 work? I tried it, and I feel like the top and bottom tile padding gets a bit weird. Maybe the text is too scrunched?
Whichever option we choose, can we decide and ship? |
@mheap @lena-larionova @cloudjumpercat I'm merging this and we can iterate again if we need to. Fwiw this looks fine to me. |
Description
Adding AI Gateway as a key feature to our docs landing page.
Changed the rows to a two-column max because otherwise it's a 3-3-1 tile overhang. Not sure if I like this more, please take a look.
Testing instructions
Preview link:
Checklist