feat: homepage refresh (deprecated 🚨) #119
Conversation
✅ Deploy Preview for opensauced-landing ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
|
I know this is still a WIP, but want to point out that the images in the preview includes the older images. We will also need to place these images local in the repo, and not on Sanity. Sanity scales the images down a ton.
cc @getaheaddev |
|
We need a mobile view of this element.
It's too small |
|
Hey @shamimbinnur, communicating here for visibility. Per our discussion: Hi-res images and updated layout/copy. I'll wait for your green light to go through both the desktop and mobile versions and consolidate my feedback to give to you all at once. Thanks! |
| <a className="text-sm text-wh pt-7" href="#"> | ||
| Working with a team? Click here | ||
| </a> |
There was a problem hiding this comment.
This should be a link to the previous landing page at the /teams route.
There was a problem hiding this comment.
Do you mean https://opensauced.pizza/teams?
There was a problem hiding this comment.
Do you mean https://opensauced.pizza/teams?
Correct
| <Heading alignSmall="center"> | ||
| $orange-to-yellowThe fast track$orange-to-yellow to open source | ||
| </Heading> |
There was a problem hiding this comment.
There are two newsletter sections and should be one. We only need email, no name as well.
There was a problem hiding this comment.
Hey @shamimbinnur, let's delete the "Subscribe for Extra Sauce" newsletter section and keep the "The fast track..." section as is (without the 'Your Name' of course). Could we also switch out the copy from "Sign Up" to "Subscribe"?
Thanks
|
Can we have the bottom copy "Working with the team? Click here" stylized like the other text? The font color and font itself is different from the rest of the body font on our site. Let's also have "Click here" highlighted in our gradient color . Also when we click on "Click here" it goes to a 404, we need to have it route to the enterprise site. @bdougie , did we want to have that opensauced.pizza/enterprise ? |
|
The mobile site still has low-res photos and need to be retina scaled. examples attached
|
I have just realized that this issue is occurring on the Safari browser. I will explore ways to resolve it." |
|
Thanks @shamimbinnur . We've also updated the repo segment on the mobile version, could you take a look and let me know if you have any questions? Also 2 things:
|
shamimbinnur
changed the title
What type of PR is this? (check all applicable)
Description
Related Tickets & Documents
Mobile & Desktop Screenshots/Recordings
Added tests?
Added to documentation?
[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?