Design comparison
SolutionDesign
Solution retrospective
Please feel free to comment and suggest to improve.
Community feedback
- @lack21Posted over 1 year ago
Great job 👍, but I have some recommendation!
- When width of the screen reaches 360px, layout gets broken, I noticed you have this
@media screen and (min-width: 360px) and (max-width: 767px)
, so I suggest you to remove(min-width: 360px)
since you already have(max-width: 767px)
there's no need for min-width.
Marked as helpful0 - When width of the screen reaches 360px, layout gets broken, I noticed you have this
- @HassiaiPosted over 1 year ago
Replace <div class="container"> with the main tag to make the content/page accessible. click here for more on web-accessibility and semantic html
For the color of the image , add a background-color of soft violet , background-blend-mode: multiply and opacity: 0.8 to .col-right.
.col-right{ background-image: url(); background-size: cover; background-color: hsl() background-blend-mode: multiply; opacity: 0.8; }
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
Marked as helpful0
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