Conversation
|
@LanesGood is attempting to deploy a commit to the Radiant Earth Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@alukach FYI, this isn't building. LMK if you're able to troubleshoot why Vercel is failing. |
|
@LanesGood there were some build errors (fixed via 322887f & 9cfa79b). You can test the build locally via |
|
@alukach thanks for the build fixes. This is getting close. FYI I did do a pretty big reshuffle to create route groups, simply to have a different layout page for the landing page, which avoids needing pathname detection in the top navigation (and ideally the footer). However, if this is overly complex and/or we want to simply update the navigation to follow the new proposed designs throughout the whole application, we could likely remove this route group update. |
|
@alukach this is ready for review |
|
This looks great 🎉 ! I've added @PowerChell and @jedsundwall for review and will defer to them for final say on aesthetics. A few things stand out to me:
Note I don't think we need to go off and make changes for all of the aforementioned thoughts. Rather, I think it makes sense for us to figure out which things should be left as is, which (if any) should be changed before merging this PR, and which (if any) should be changed in another PR. |
|
Looks amazing, @LanesGood! And thanks for starting the review like this, @alukach! I'm off this week. Very open to comments from @PowerChell in the meantime. I'll add my review (maybe my own PR) next week, hopefully while en route to DC! |
|
Thanks for the initial feedback @alukach. A few quick responses:
|
|
This needs approval for redeployment on Vercel. I've fixed merge conflicts. Have looked through @PowerChell's review above, and did remove the duplicate section headers mentioned in item 2, but will assume I should wait on further discussions or signal on go-ahead from @jedsundwall and anyone else before implementing other changes to content, layout, etc. |
|
@alukach and @LanesGood I promise I'm going to look at this on Monday. Can either of you see why it's failing on deployment (I'm pretty sure it's not failing because it needs Vercel auth)? |
| :root, .light, .light-theme, .radix-themes { | ||
| --color-background: #edecea; | ||
| } |
There was a problem hiding this comment.
This needs to be moved to globals.css for the project to build successfully. Otherwise, we get:
Syntax error: Selector ":root, .light, .light-theme, .radix-themes" is not pure (pure selectors must contain at least one local class or id)
| .heroImage { | ||
| mix-blend-mode: difference; | ||
| } |
|
@alukach agree with you on the globe mix mode. On the global styles, strangely this build error doesn't present locally for me. Keeping these variables on the landing page module is what allows me (on my local build) to see distinct color palette styles for the homepage vs the rest of the pages:
|
- Update hero section copy to emphasize "Data Publishing Utility" - Rewrite "Why Source" section with clearer challenge/solution framing - Reorder feature cards and update pricing messaging - Simplify footer by removing impressum section - Fix globe image overflow by adding clipping to landing container - Increase carousel arrow size for better visibility Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
jedsundwall
left a comment
There was a problem hiding this comment.
This is solid. Further comments coming on the PR to refine our look and feel over the coming weeks.
|
Hi team: I think this is good to push live as it is now. A few notes on what I'd like us to work on separately in the future. We can probably create issues for these.
In the meantime, @alukach can you please help us get this live asap? Today if possible? |













Update landing page styles for Source.coop
Todo