@angelveliz1837Submitted 3 months ago
Latest solutions
Article Preview Using React + Typescript
#react#typescriptSubmitted 3 months agoI know I need to edit the the toggling active links (I think the solution would be to pass state through a parent, but I have yet to master that!). I also need to add the triangle to the bottom of the share link box in desktop view. Would love help/advice with either!
Latest comments
- @savvycolleenPosted 3 months ago
Hi! The screenshot of your solution looks great! What stood out to me most when I previewed the site is the responsiveness. The desktop version and mobile version look great, but going from desktop to mobile looks a little wonky. It looks like you have the media query at a max-width of 375px, but as soon as you get to that point, the desktop version doesn't have room, so there is no room for it to expand. Maybe you could play with that number and make it bigger until there is room for it to expand. Just a thought! The toggle function you have is great! Well done.
0 - @MrSeagerSubmitted 10 months ago@savvycolleenPosted 9 months ago
Loving the scroll effect!! So cool. Everything looks really good. There are no complaints with your outcome! The only constructive feedback I can think to give would be possibly breaking up your RecipePage.js into components for readability. Cheers to you!
Marked as helpful1 - @TarteelAkSubmitted 10 months ago@savvycolleenPosted 10 months ago
Good afternoon! Thanks for sharing your solution--the card looks really great! In looking at your styling, I love how you used calc for the width of your container. That is a great idea that added to my thinking! Generally speaking, your design turned out bigger than the original, so sizing some things down like the size of the image, h1, and the margin tops of your social links could help make it look even closer to the original (if you are trying for a more precise solution!). Hats off to you!
0 - @ahmeteidSubmitted 10 months ago@savvycolleenPosted 10 months ago
You card looks great! I love that you used variables for your colors in CSS. I always intend to do that and then skip that step. It's a great habit to get into it! I also like how you used "fit-content" for the yellow background on "Learning"--that is probably more simple than what I did in my solution. The only thing I noticed for the html is putting your divs in something more semantic like "section" or "main." I know it feels weird when there is not a full layout, but it seems like a good practice to follow when possible! Cheers on your work!
0 - @E0MRSubmitted 10 months ago@savvycolleenPosted 10 months ago
I love that you centered this by using flex on the body! A creative solution and great idea. Hats off to you!
1
