Mixed-Messages: academy project - random advice

Projects Portfolio Project: Mixed Messages
1

Hi,

My project was inspired by a party game.
I purposely tried to keep the code as simple as possible. Most of my time when into finishing touches with the HTML and CSS, The course hasn’t covered them yet, so there was a lot of trial and error involved.

Any feedback will be appreciated, no offences taken :wink:

1 Like
2

Welcome to the forums!

This is great, well done! :raised_hands:

It’s really cool that you’ve incorporated HTML & CSS into it before those have been taught yet, great job :smile:. The only (very fussy!) bits of feedback I would give are as so:

  • <div id="main"> could be replaced with a <main> tag which is better for accessibility (for example via screen readers) as it’s more semantic and better describes the purpose of the tag.
  • It might be a good idea to replace jcript_mm.js with a slightly clearer file name, something like script.js is pretty common.

That said, that’s me being really overly picky and it’s awesome, congrats!

3

Awesome project!!! Love it! :smile: :+1:t3: :+1:t3:

4

Thanks for the greetings, compliment, and special thanks for the feedback.

*Regarding the HTML, I didn’t know better. I’ll change that after I have read into it and understand what it means.

*With the strange file name, it is a habit that I should try to kick,

Thanks again

1 Like
5

Thank you, much appreciated.