Skip to content
OverviewSolutions
6
Comments
6

DaniAe

@DaniahAb60 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

Latest solutions

Latest comments

  • DaniAe•60
    @DaniahAbPosted 2 months ago

    Looks really good!

    0
  • DaniAe•60
    @DaniahAbPosted 2 months ago

    It looks great. nice job!

    0
  • DaniAe•60
    @DaniahAbPosted 3 months ago

    Nice work

    1
  • DaniAe•60
    @DaniahAbPosted 3 months ago

    Some differences from the design. maybe consider moving the styling of the image to the CSS file. But overall looks pretty good. Nice work!

    0
  • Mohammed-Rishin-M-B•50
    @Mohammed-Rishin-M-BSubmitted 3 months ago
    What specific areas of your project would you like help with?

    how do you place the images inside the div with equal margin spaces.in mine if i change the size my image is going outside the div.it is not proportionally shrinking with the div. how do you do it?

    blog-preview-card

    2
    DaniAe•60
    @DaniahAbPosted 3 months ago

    Not sure exactly, but try removing this part of the code:

    @media(max-width:540px) {
    .box {
        width: 20rem;
        height: 30rem;
        
    }

    .image-main {
        height: auto;
        width: 80%;
        margin: auto;
        border-radius: 8%;
        
    }

    And making the width of .image-main 100% (to fit the .box div) and change the height in .box to max-height. I think it solves the problem. Other than that, good work!

    0
  • DaniAe•60
    @DaniahAbPosted 3 months ago

    Good solution with the use of flexbox and code readability. Could improve accessibility by adding alt text to describe the image. And you could combine the two "p" elements in the CSS file since they're targeting the same text.

    0