carlwoodwiss
commented
6 years ago Thank you for the feedback. It is all very helpful and very much taken on board.
One thing I will say is that you have referenced using things that we haven't learnt yet (responsive design for example - that is a part of the program that is coming later up so I haven't learnt how to do this yet.)
Missing semicolons - just a silly mistake and you are correct, probably from copying and pasting.
Never heard of the W3C validator as it is not part of the course (unless it comes up later). I will try to learn how to use it but it would be really helpful if there was a part of the course explaining it.
I have also taken on getting more specific with id allocation. Makes a lot of sense and I am sure it will help.
I really like that you commented your HTML and I would recommend extending this to your stylesheet. While this is a smaller webpage in larger web pages this will seriously cut down on the amount of time you spend debugging the Graphical User Interface or webpage when you have descriptive comments to guide you. I highly recommend ensuring that you are testing your page in various browsers, this will ensure that users visiting your web page have a great user experience and will want to return. Along with responsive testing, make sure to validate your HTML using W3C Validator. This validator is a great tool to validate your HTML and CSS and ensures you stay current with changes and additionals and also write quality code. Keep up the really great work and never stop learning more as you continue your journey into web development.
Meets Expectation