Design comparison
Solution retrospective
I couldn't figure out the color styling on the main title in the hero section. Can anyone help?
I haven't made an attempt at the lazy-loading effect for the images. I'd like to try that out and re-submit this challenge.
Community feedback
- @farukingPosted over 2 years ago
- You can use 'span' element to style the needed letters that should be in black.
- The heading in the last box (come and be inspired) should have a vertical space between it.
- Design is not yet responsive.
0@mseidel819Posted over 2 years ago@faruking Thank you for the feedback! Can you tell me what browser you’re using to view my page? It’s responsive for me on my iPhone and mac. I haven’t checked pc. Do you have any more details on what is breaking?
0@vanzasetiaPosted over 2 years ago@mseidel819 Hi there! I can try to explain some of the issues with the responsiveness.
- On the home page
- If you see the site on
600px
width, the layout for the second section should have full height. I mean the images should have full height on it. I don't know React but, you should use grid for this type of layout. - The
footer
also needs more space so that the paragraph element (The Modern Art Gallery is free to all visitors...) is not too narrow.
- If you see the site on
- In general, for both pages, don't have a container limit in the width. As a result, on a wide screen (such as
2000px
width), everything almost become one line.
The "MODERN ART GALLERY" text on the home page (the first one), it's different from the design. So, I recommend trying to create the effect where the text color is adapted through the different background colors. You can read this CSS Tricks article where it explains on how to do that.
I am on Windows 10 and Chromium browser base, specifically Brave. I am not sure why you need such information.
Marked as helpful0@mseidel819Posted over 2 years ago@vanzasetia thanks for clarifying! I wasn't sure just how much of the page wasn't changing to screen sizes.
Thanks for the CSS link too. I had a lot of trouble with that so I ended up leaving it as-is. Now I can update it.
0@vanzasetiaPosted over 2 years ago@mseidel819 No problem! I'm glad to help you! 😄
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