Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Music Maven- My music Quiz Bot #288

Open
wants to merge 19 commits into
base: main
Choose a base branch
from

Conversation

Anna2024WebDev
Copy link

Copy link
Contributor

@HIPPIEKICK HIPPIEKICK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ha, what a superb idea for a chatbot! 🎵 I got 6/8 🥳 Let's have a look at some feedback.

JavaScript

  • Well-defined DOM selectors and global variables!
  • Storing questions as objects and placing them in an array is a great way to manage the quiz data. This allows for easy expansion of the quiz by simply adding new question objects to the array 🥳
  • The logic flow for the quiz works well 👍

Clean Code

  • Remember to follow JS naming convention and name all your variables with camelCase.
  • Be consistent with whether you're using double or single quotes

Quite advanced flow, a bit out of scope so it's no wonder you feel like you have to recap what you've done. Hopefully it will clear up a bit in the coming weeks. Keep up the good work ⭐

code/script.js Outdated

// Functions goes here 👇
let currentQuestionNumber = 1 // Start with Question 1
let score= 0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let score = 0*

Comment on lines +207 to +240


































Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ending with one empty line is enough 😉

Copy link

@joheri1 joheri1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What a great idea to make a quiz chatbot! I like that you made your own design, with new images for the user and bot. The code structure is clean and easy to follow, the comments make the code even easier to follow. The functions askQuestion and checkAnswer are written in a way that can be reused for each question, which is excellent.

The only thing I noticed that can be improved, is that the chat works like a charm on desktop, but on a mobile I need to scroll to see the options for each question. The media queries in the CSS are a bit unstructured so it's hard to see what does what.

Overall, I think your chatbot is awesome! Well done, Anna!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants