ysabelm / prj-rev-bwfs-dasmoto

0 stars 0 forks source link

SUMMARY #4

Open blerude opened 6 years ago

blerude commented 6 years ago

Grade: Meets Expectations

Awesome job taking this project on! You've included all the necessary elements and your code is easy to read through. You have CSS down--it's efficient and effective. You have a great sense for how to apply classes and ids as well.

Moving forward, I recommend that you pay attention to your formatting. Div wrappers are a useful formatting tool for showing how elements relate and for generally organizing your code. And making sure that your alignment makes sense is important as well. You should add comments to further clarify the purpose of each block within the larger project. Finally, feel free to take liberties with your CSS to make your project more visually engaging. A few resources to help you do that are Google Fonts for finding cool fonts, and Coolors.co, which generates free color palettes to implement in your design. You might find those fun to play with, and they'll help you infuse your apps with your own style. Again, great job and congratulations!

ysabelm commented 6 years ago

Thank you for your feedback. You're right, I used in a wrong way the indentation. I'll be more careful. As for div wrappers, I understand it's an important tool I have to handle better.