Jonathan• 510
@corjon
Submitted
All feedback is welcome. What do you think about radio buttons for this challenge? I'm curious to know what solution other people used. Thanks!
Brendan
@brendanmadden
All comments
- Brendan• 170
@brendanmadden
Posted
This looks awesome, Jonathan! Great work! It looks like you were able to complete the challenge with a lot less code that I was, so I'll have to take some notes!
It looks like you may have included an extra period in the file path for your star svg, so it's not showing right now => <img src="images/icon-star.svg." alt="">.
Also, maybe you left it there intentionally, but you could remove the border from your submit button, and everything would look just like the design!
Awesome job! 🎉
Marked as helpful
1 - dewslyse• 3,025
@dewslyse
Submitted
Hi guys! Just completed another challenge 🚀. I really enjoyed this one. I sneaked in an extra rating card ⭐ and a testimonial card for mid-sized devices 😁. Lemme know what you guys think. 👍 or 👎
Brendan• 170@brendanmadden
Posted
Looks really, really, good, @dewslyse! 🎉 Nice work! I think I could learn a thing or two by looking through some of your code! Nice fluid responsiveness, and I love the Dews Original card in the tablet view 😂 Looking forward to seeing what you build next!
1 - CWu• 270
@cwu90
Submitted
This is my first project here at Frontend Mentor. Any feedback is appreciated since I know I have much to improve on with my responsive design and html code. Thank you!
Brendan• 170@brendanmadden
Posted
Awesome work, CWu! Looks great. Look into the CSS "mix-blend-mode: multiply" property, and you might be able to get that color just right!
Good luck!
0 - Joseph• 770
@Jos02378
Submitted
Hi! I want to ask several question about my solution.
-
is it better to code the mobile pages first then the desktop pages, or is it better to code the desktop pages first then the mobile pages?
-
Is there any suggestion on how to make the card to be responsive?
Thank you!
Brendan• 170@brendanmadden
Posted
Hey Joseph!
Nice work on this. It looks responsive to me! I notice though that there is a red border on the wrapper after the media query!
Also, in order to get the color right on, you might want to try the "mix-blend-mode: multiply;" property!
I hope this helps!
Good luck!
Marked as helpful
1 -
