reeceds / prj-rev-bwfs-dasmoto Goto Github PK

Your lines are a bit long right now. Try splitting this into new lines so it's easier to read on one screen without scrolling. Convention is usually ~79 characters per line.

https://github.com/Reeceds/prj-rev-bwfs-dasmoto/blob/master/Dasmoto'sArts%26Crafts/index.html#L31

Good job specifying the span css selector class and good choice of name. Clear and concise names are the best. This gives you more flexibility as your code grows in size and makes your code more reusable since if you need to do similar highlighting in the future, you can just reference this again.

github.com/Reeceds/prj-rev-bwfs-dasmoto/blob/master/Dasmoto'sArts%26Crafts/index.html#L17

Try to save your images locally instead of adding a link. Saving images adds to the stability of your website because your website is not reliant on the link to be valid all the time. So next time, save all the images under resources/images/

https://github.com/Reeceds/prj-rev-bwfs-dasmoto/blob/master/Dasmoto'sArts%26Crafts/index.html#L15

Summary Grade: Exceeds Expectations

Good job with your code! I really like the use of indentation and whitespace in your html code! Makes your code very easy to read! Also, good decisions on the classes and ids name. It makes it very clear what its purpose is and makes grouping your sections very effective. These organizational skills come in handy as you expand your code. Great start to the Ready program!

I included a few notes in both the html and css files for you to think more critically about. First, make sure to use the html & sign instead of just &, which is &. This is just a convention. Finally, your lines are a bit long right now. Try splitting this into new lines so it's easier to read on one screen without scrolling. Convention is usually ~79 characters per line.

Try adding comments to your css page. Unlike html, css code is not naturally defined by its structure. So having comments helps you organize it and quickly navigate it. As a suggestion, add comments at the start of each section of the corresponding html code so readers can follow clearly between the two files. This will become very useful in the future as your code grows and your css becomes more complicated.

https://github.com/Reeceds/prj-rev-bwfs-dasmoto/blob/master/Dasmoto'sArts%26Crafts/resources/css/index.css#L1