kellyskarritt / MI-449-SS17-740-css-intro-iMnt09

0 stars 0 forks source link

Project Feedback #1

Open kellyskarritt opened 7 years ago

kellyskarritt commented 7 years ago

@chrisvfritz Can you take a look at this? It's hosted here and meets the following criteria:

chrisvfritz commented 7 years ago

Wow, that was quick! 😄 I haven't looked at the CSS yet, but I do see one area where I think we can get a little closer to the mockup: in the spacing between images. Below, you'll see that your version (on the left) has quite a bit more space between images than the mockup (on the right):

Mockup difference

kellyskarritt commented 7 years ago

Fixed it! thanks

chrisvfritz commented 7 years ago

Beautiful! 🎉 👍 :shipit: