Skip to content
OverviewSolutions
4
Comments
4
P
galonaranjo
@galonaranjo

All comments

  • galonaranjo•80
    @galonaranjoPosted 11 days ago

    The design and details of the content are pretty spot on, Mariam. Not sure if this was by design, but it looks like the entire card might be set to fit the viewport on desktop, shrinking all the content to fit into it.

    Also, some small details might need to be made to the image as the responsive design changes. This was a bit tricky for me, but changing the container's padding on mobile view so that there is no padding around the image.

    But overall it looks great!

    Marked as helpful
    0
  • galonaranjo•80
    @galonaranjoPosted 3 months ago

    Hey Tony, Looks like you might have submitted something different from the solution.

    0
  • galonaranjo•80
    @galonaranjoPosted 4 months ago

    Looks great overall! Looking at the side-by-side, it seems like the card container could have been just a bit bigger. You can also remove the third value from the box-shadow property so there is no blur effect.

    box-shadow: 5px 5px 3px black; → box-shadow: 5px 5px black;

    0
  • galonaranjo•80
    @galonaranjoPosted 4 months ago

    Great use of presetting the colors up at the top. It makes the stylesheet look a lot cleaner.

    Marked as helpful
    1