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

Will you survive the zombie apocalypse? #446

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

SaraBat
Copy link

@SaraBat SaraBat commented Mar 19, 2023

Copy link

@AntoniaGranit AntoniaGranit left a comment

Choose a reason for hiding this comment

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

I loved the quiz! It was super fun and quite complex, you had a lot of different steps but still kept the code neat and easy to read. The styling is also really good. I personally would have maybe split up the CSS file into smaller files corresponding to their respective components, but I don't think the index.css-file is that long to begin with. Good job!

<Footer />
</>
);
} else {

Choose a reason for hiding this comment

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

This is a good way to build up the quiz, I like that it gives you an update about the apocalypse in the middle of it.

</>
);
}
}

Choose a reason for hiding this comment

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

The code of how the quiz progresses is well-organized and neat.

- watch <em>Pride Prejudice and Zombies</em> daily for the next month.
</p>
</section>
);

Choose a reason for hiding this comment

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

This was such a fun summary, and you kept track of all the results of the quiz in it as well!

gap: 20px;
border: 5px double black;
border-radius: 25px;

Choose a reason for hiding this comment

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

I love the layout of the quiz

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.

2 participants