Design comparison
Solution retrospective
Hey Everyone this is my second challenge done here on Frontend Mentor. I thought this would be easy but oh boy I was wrong. One thing I realized while doing this project was I don't know CSS as much as I thought I did. I know I had to cut a lot of corners by using margin and padding. I also don't know how to create it mobile size yet. I'm learning that now. Any feedbacks would be much appreciated. Also, the background color of the image wouldn't change for me. Any suggestions? Thank you. I'm already loving this community and I have learned a lot from the feedback I got on my first challenge!
Community feedback
- @st3wnPosted about 3 years ago
Hey man,
for the colored image you can use a linear-gradient. Example:
background-image: linear-gradient(rgba(170, 92, 219, 0.45), rgba(170, 92, 219, 0.45)), url("../img/image-header-desktop.jpg");
In this case, the image would have a pink overlay (0.45).
Marked as helpful1@yaredh30Posted about 3 years ago@st3wn Hey man thanks a lot for the suggestion on how to change the image color. I appreciate it. Ill give it a try! Can you view my code?
0@st3wnPosted about 3 years ago@yaredh30 its late for me so i will just use the screenshot
-use text-align: left; instead of text-align: center; -use linear-gradient as I said -the words on the bottom have to be in CAPS -use flex box -use font-weight: bold; on the words 10k, 314, 12M+ -give the words 10k, 313, 12M+ a line-height: 0; to set a margin between this and the text under it -check this challenge on my profile and check my code for some isnspiration ^^
0@yaredh30Posted about 3 years ago@st3wn I tired using the display: flexbox for the words and nothing changed. I also tried changing the image color like u told me but my image is gone now. I'll deff check out your work!
0 - @mdsameer1992Posted about 3 years ago
Hey. Couldn't check your code. Quick review. You didn't make a responsive design for mobile. For image color I tried overlaying another div element on top of image with color given in styleguide and apply hue saturation contrast and opacity.
Marked as helpful1@yaredh30Posted about 3 years ago@mdsameer1992 Why couldn't you check out my code? I think its because maybe its at the bottom. Yea I'm still learning how to make a responsive design for mobile. I'll give the image color a try. Thank you for your feedback. Means a lot!
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