Seeking Constructive Criticism on My HTML/CSS/JS Personal Website
Asked by
CuriousCoyote72
I'm in the process of rebuilding my personal website and I decided to share the code publicly instead of trying to figure everything out alone. This is an actual site that I actively use, not just a project from a tutorial. I'm focused on tidying up the structure, enhancing the CSS, and improving the JavaScript, while ensuring I'm not making any mistakes in terms of performance or accessibility. I'm not a professional developer; I'm a 44-year-old guy from West Virginia, learning and fixing things as I go. If you notice anything that's unclear, messy, or just wrong, please let me know. I appreciate any honest feedback! Here's the link to the code repository: [github.com/lessofjosh/lessofjosh-website](https://github.com/lessofjosh/lessofjosh-website)
Answers
You should consider combining your CSS files into one that you can link to from each HTML page instead of repeating styles. The same goes for your JavaScript, if you have multiple files. Also, running your JavaScript through a Beautifier tool can help improve readability. These are just suggestions for now, but it's a great start! Keep it up!
Reply to this answer
One of the first things I'd recommend is focusing on code indentation for better readability. Using proper indentation helps your brain parse the code structure quickly and shows the hierarchy of elements. Also, I noticed some optimization issues with images; you're loading larger images than needed. Adding alt tags to images and names to buttons is crucial for accessibility too. Your robots.txt also seems to have some errors.
Reply to this answer
It's awesome that you're using plain HTML, CSS, and JS—great for learning! One area to improve is your form elements; they lack labels and solely rely on placeholders, which isn't user-friendly. Adding labels can really enhance usability.
Reply to this answer
I think you could benefit from asking DeepSeek to optimize your code. I’ve made several sites with their help and they can be really resourceful.
Reply to this answer
If you're interested in data binding, you might want to explore using Vue. It's totally fine to stick with vanilla JavaScript for learning, but using Vue can simplify your custom JS. Just remember, organizing your CSS into separate files can help with performance through page size and caching. Nice to see you're incorporating CSS grid!
Reply to this answer
Add your answer
Answers are reviewed before publication.
Related Discussions