@Eric-Ferole
Posted
Great job ! It look pretty much as the design. Few littles things that are almost invisible but I thing I worth doing it.
- The margins of your .title are not accurate. Think of resetting some the CSS default value by adding a CSS reset (
* { margin: 0; }
) and then adjust every margin manually to gain more control. Here is the one I use : My Custom CSS Reset - Put the margin on the outer block instead of inner element. Per example, the <p>s of
.additionnal-info
have margins on them. The margin should be on.additionnal-info
. - Instead on adding a fix height on your .image you can add a display: block; on your image to make your image fit in the
.image perfectly
. - You should not use fix height for your components. By doing so, you prevent the div to expand if we add more content dynamically. The box size of your block plus the margins should be all you need.
Hope it helps. Keep up the good work ! :)