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
@brendanmaddenAll comments
- @corjonSubmitted about 2 years ago@brendanmaddenPosted about 2 years ago
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 helpful1 - @dewslyseSubmitted over 3 years ago
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 👎
@brendanmaddenPosted over 3 years agoLooks 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 - @cwu90Submitted over 3 years ago
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!
@brendanmaddenPosted over 3 years agoAwesome 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 - @Jos02378Submitted over 3 years ago
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!
@brendanmaddenPosted over 3 years agoHey 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 helpful1 -