-
Notifications
You must be signed in to change notification settings - Fork 1
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
Research + Template feature pages #59
base: main
Are you sure you want to change the base?
Conversation
96bb2e0
to
24bd1f3
Compare
Deploying prem-website with
|
Latest commit: |
0dc7f1b
|
Status: | ✅ Deploy successful! |
Preview URL: | https://b2856ffc.prem-website.pages.dev |
Branch Preview URL: | https://feature-new-pages-02.prem-website.pages.dev |
48baa21
to
bfb5d26
Compare
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.
can you remove Lorem ipsum, rest looks good.
|
||
<p class="h2 models-slm__title js-anim-item">Prem's Specialized SLMs</p> | ||
<p class="models-slm__text js-anim-item"> | ||
A Suite of Foundational Open Source SLMs to Supercharge Business |
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.
@filopedraz can we review if the copy is good here, I mean how fonts are capitalised, looks good to me, but from a copy standpoint.
another design question @filopedraz, should this entire image not be the thing user should see? like this Screen.Recording.2024-06-03.at.7.02.12.PM.mov |
bfb5d26
to
0dc7f1b
Compare
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.
looks good, let's just wait for @filopedraz to review once
NB:
template-feature.html
is a page template, you can use it to create page templates of all features by making a copy and changing the content.