RatifiedGeorge• 630
@Ratified
Submitted
Any advice on the color overlay of the image, I couldn't manage to make it look exactly like the one on the design. Looking forward to your comments on how I can improve that.
MICHAEL
@Generoustechnocrat
All comments
- MICHAEL• 150
@Generoustechnocrat
Posted
Hello George, you did great but you can try this and it will look good background-color: hsl(277, 64%, 61%); background-blend-mode: multiply;
1 - Khazar• 50
@Xezer669
Submitted
I had problems with the grid system and flexbox. And I'm not too sure about the mobile version.
MICHAEL• 150@Generoustechnocrat
Posted
Hello KHAZAR you did great, you just need to spice things up to make it look good. I try viewing your code but it seems you didn't put the right site. try this out on your image container.....background-color: hsl(277, 64%, 61%); background-blend-mode: multiply; i hope this feedback is helpful happy coding.
0 - RobertoMada• 220
@RobertoMada
Submitted
Hello guys, can you give me feedback about what can I improve or what I
m doing wrong? I struggled with the size of the background image, I couldnt find a solution to display it better.MICHAEL• 150@Generoustechnocrat
Posted
Hello ROBERTOMADA you did great but you just need to make little correction to get it right. Here is what i suggest you do. you need to create separate div for card and the image-container you also need to create a div container that holds both the image-container and card. you can use this properties to center your div: min-height:95vh; display:flex; flex-direction:row-reverse; align-items:center; justify-content: center; padding:10px; make sure you use the same width and height for image-container and card, width:440px; height:335px; border radius for the card is border-radius:9px 0 0 9px; while the border radius for the image is border-radius:0 9px 9px 0; I believe you are good with every other things. I hope you find this comment helpful. happy coding.
Marked as helpful
1 - Emmanuel Akpan• 790
@heisemmaco-dev
Submitted
all comment are welcomed, but i have a little bin of challenge with switching between the desktop pic and moblie pic.
MICHAEL• 150@Generoustechnocrat
Posted
Nice work you got here bro, i toggled the design and solution and i think that there are few things to fix. check your style-guide and stick to it.
Marked as helpful
1 - SAHULHAMEED U• 160
@Sahulhameedu
Submitted
Please tell me how I can improve
MICHAEL• 150@Generoustechnocrat
Posted
Hello @Sahulhameedu nice work, but there's a room for improvement, I will advise you stick to style guide that was made available. You didn't use the font-family, I will advise you replace the div tag with main tag for easy accessibility, <h4> with <h2> Img{ max-width: 100%; Border-radius: 15px; Padding:5px; } Work on the body instead of the main tag Body{ min-height:100vh; Display:grid; Place-items:center; } You didn't add color to you p tag, do that as instructed and I hope this is helpful, Happy coding.
Marked as helpful
0
