Skip to content
OverviewSolutions
7
Comments
7
MICHAEL
@Generoustechnocrat

All comments

  • RatifiedGeorge•670
    @RatifiedSubmitted over 1 year ago

    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.

    Stats preview card component

    1
    MICHAEL•150
    @GeneroustechnocratPosted over 1 year ago

    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
  • MICHAEL•150
    @GeneroustechnocratPosted over 1 year ago

    Good work Rocio, you can try this out...background-color: hsl(277, 64%, 61%); background-blend-mode: multiply;

    Marked as helpful
    0
  • MICHAEL•150
    @GeneroustechnocratPosted over 1 year ago

    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
    @RobertoMadaSubmitted over 1 year ago

    Hello guys, can you give me feedback about what can I improve or what Im doing wrong? I struggled with the size of the background image, I couldnt find a solution to display it better.

    Responsive Stats Preview Card

    1
    MICHAEL•150
    @GeneroustechnocratPosted over 1 year ago

    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
  • MICHAEL•150
    @GeneroustechnocratPosted over 1 year ago

    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
  • MICHAEL•150
    @GeneroustechnocratPosted over 1 year ago

    The price was heck of a problem, I don't really know why the fonts ain't working there, I did all i could do, yet still looks that same way.

    0
  • MICHAEL•150
    @GeneroustechnocratPosted over 1 year ago

    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