Add C to Rust Migration course landing page#2865
Conversation
|
The slug in the PR description wasn't updated from the previous version (now it is), it should be /c-to-rust-migration/, so the working preview is here: https://deploy-preview-2865--objective-northcutt-37494c.netlify.app/c-to-rust-migration/ I still need to address Linn's latest comments from Notion though. |
|
I think it looks good! |
marcoow
left a comment
There was a problem hiding this comment.
"Your Mentor" makes sense for workshops but "Your author" sounds a bit strange
Co-authored-by: Marco Otte-Witte <marco.otte-witte@mainmatter.com>
Co-authored-by: Marco Otte-Witte <marco.otte-witte@mainmatter.com>
Co-authored-by: Marco Otte-Witte <marco.otte-witte@mainmatter.com>
marcoow
left a comment
There was a problem hiding this comment.
I think the "by Mainmatter" needs to be closer to the headline (like it's here: https://rust-exercises.com) – otherwise it looks strange:
Should we have some kind of grid for these links so they don't look as unaligned?
I think it would be nice to have a larger image of Jonas (I know this is an existing component but still):
Is the arrow not on the wrong side?
I'll look into copy on Friday
|
@marcoow I added the Mailchimp audience "Mainmatter C to Rust", so please use that for the sign-up form. I set it up as a duplicate based on the Rust audience, feel free to make any changes you see fit directly in Mailchimp. |
|
@avollrath @mainmatterlinnbp we should think about where and how this page is linked from the rest of mainmatter.com. If you get to this and navigate away, there should be a way to find it again? |
Co-authored-by: Marco Otte-Witte <marco.otte-witte@mainmatter.com>
It needs to be in the secondary nav, together with Rust exercises. We should add a "Learning material" or such. We might need a "Learning material" landing page for it. |
015bd93 to
a673143
Compare
|
@avollrath @marcoow I pushed a change with my suggestions for improved copy, please have a look! |
|
Thanks @JonasKruckenberg. Looks good to me overall, I'll make the copy a bit smoother next week. |
61163ba to
9056487
Compare
|
@avollrath can you finish the above task for you? |
marcoow
left a comment
There was a problem hiding this comment.
@avollrath I think it would be nice to keep the book illustration on mobile (I understand we want to save space but at least a small version would be good):

Summary
Adds a new landing page for the C to Rust Migration course at
/c-to-rust-migration-book/.TODO
og:image