Design comparison
Solution retrospective
10/21/20
I'm having issues with stretching the background image all the way through.
Honest feedback/criticism is appreciated! Any suggestions on how I can improve this solution?
Update 10/21/20
- Fixed the background issue after countless tweaks.
background-position: left top, right bottom;
Community feedback
- @ApplePieGiraffePosted about 4 years ago
Hey, good job, Dwight! 👏
Your solution looks good and it's responsive, too! (And I see you managed to position the background SVGs. 😆)
I only suggest,
- If you are to break into a mobile-friendly layout rather quickly (
1285px
in your solution), setting amax-width
on the review boxes containing the SVG stars and the testimonial boxes might be a good idea to prevent them from appearing too wide when the layout first changes.
Keep coding (and happy coding, too)! 😁
1 - If you are to break into a mobile-friendly layout rather quickly (
- @PhrukeyPosted about 4 years ago
Hi SarsiPC, I'm glad you fixed the background issue. I think a background-size: cover should also work in this case. Cheers
1@SarsiPCPosted about 4 years ago@Phrukey That would work if the background was one piece. But in this solution, 2 parts of the background was given. You had to place them in the corners.
0@luwa-starPosted about 4 years ago@SarsiPC Actually, you can. Just add it to the background shorthand for both SVGs but you'd have to consider how it would look.
0 - @luwa-starPosted about 4 years ago
Great job! It looks really nice.
However, it looks nice on specific screens. I suggest you make better use of the media queries. If you choose to only consider the screen sizes in the
style-guide.md
. Your design won't be responsive on other screen sizes. Instead of usingmax-width:1280px
try going formin-width:1280px
. Also, choose a suitable min-width for medium screen sizes.Secondly, for screen sizes with width less than 320px there's an offset. I suggest you check that out it might be that you are using the wrong unit or there's an extra padding somewhere. You need to check it out. Btw I don't really understand why you are using min and max within the same class. Is it that there's a concept behind it? From what I know CSS interprets code in order. Don't be offended, I genuinely don't know.
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord