Design comparison
Solution retrospective
Any feedback is appreciated, especially the feedback regarding responsiveness.
Community feedback
- @ApplePieGiraffePosted almost 4 years ago
Hey, isimeri! 👋
Nice work on this challenge! Your solution looks good! 👍
I suggest,
- Adding a
max-width
to the main container or wrapper to prevent the design from looking too stretched on extra-large screens. - Allowing the design to fill up the entire area of the screen when the page is resized (so that there isn't any blank space to the sides of the content when the width of the screen decreases).
- Don't forget to add an
alt
attribute to your images and to identify each<section>
tag that you use with a heading to clear up your solution report and make your solution more accessible.
Keep coding (and happy coding, too)! 😁
0@isimeriPosted almost 4 years ago@ApplePieGiraffe Thank you for the suggestions. Totally didn't think about super big screens, gotta fix that. I don't like the extra blank space either, but the background image(family photo) looks kind of blurred already. It'd look quite ugly if i let it take up even more space, i already tried. Do
<section>
s really need headings? There is a<section>
in my code that i used just for two background images and could totally do without a heading. I guess i'll just replace it with a<div>
.0@ApplePieGiraffePosted almost 4 years ago@isimeri
You could transition to a mobile-friendly layout but not change to the mobile-friendly background image until the screen width gets even smaller. That way, the image will still look clear and only change when the width of the screen gets small enough for the resolution to not be noticed.
Also, yeah, it's recommended to only use a
<section>
tag if you have a heading to go along with it. But I think a<div>
will do fine, then, as you said! 😉0 - Adding a
- @MasterDev333Posted almost 4 years ago
Great work @Isimeri. Your solution is spot on. I have some suggestions. First, social icons are showing on my side. some problem while loading svg icons? Second, it would be better if we show footer links in 2 columns rather than in one column. Hope it helps. Happy coding~ :)
0@isimeriPosted almost 4 years ago@MasterDev333 Thanks for the suggestion regarding the footer links. I assume you meant it for when the screen width is about 600-1050px and there is enough space for a second column of links in the footer. About the social icons, what is the issue, i didn't quite understand? I used the svg's as
mask
, to be able to change their color on hover. Is this somehow relevant to the issue you encountered?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