Dylan• 290
@dquinn089
Posted
Fantastic work! Your solution it practically identical, and your code it very well structured and written! In all reality there really aren't any issues with your solution. The only thing I would recommend is making use of the different elements that HTML has to offer, to create "landmarks" in your code, making it much more readable to other devs. For example: Instead of:
<body>
<div class="container">
do:
<body>
<main>
<div>
You also have not included responsive design in your CSS, so your entire "container" element is being cut off on the left side of the screen. You can make any necessary adjustments to any of your element's properties by using CSS media queries which can be implemented at the end of your css like this:
@media () {
body {
margin:
}
main {
padding:
border-radius:
}
Hope this helps, happy coding!
0
