Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Clean code #57

Closed
wants to merge 3 commits into from
Closed

Clean code #57

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Aug 29, 2023

With these 3 commits, I did a lot of things (it took me all day 😊 ) and I solved two issues:

  1. First I put the css code and the background image of the game in a folder that I called "css".
  2. Then I created a "main.js" file where I grouped all the javascript code that was present in the "index.html". I placed it in a "src" folder with the "stats.js" file (already present in the project when I started working).
  3. To make it work, I did import/export and I activated the "module" mode on the "script" tag that I created in the html file.
  4. Finally I went through all the files to put all indentations in order.
    Everything worked when I tested it. 😃 Feel free to tell me if something is wrong.

…ed in the css folder. I also put the image in the css folder
… the standards so that the code is more readable
@ghost ghost closed this by deleting the head repository Sep 1, 2023
This pull request was closed.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant