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

Week 11 - Project Labyrinth - Yia #6

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

Conversation

Citronskal
Copy link

Netlify link

https://project-maze-yia.netlify.app/

Collaborators

solo

Copy link

@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.

Good job with the project this week 💯

@@ -0,0 +1,77 @@
import styled from "styled-components";

Choose a reason for hiding this comment

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

Ohh, I like it 😉

let imageUrl =
"https://images.unsplash.com/photo-1550100136-e092101726f4?q=80&w=2787&auto=format&fit=crop&ixlib=rb-4.0.3&ixid=M3wxMjA3fDB8MHxwaG90by1wYWdlfHx8fGVufDB8fHx8fA%3D%3D";

const imageMap = {

Choose a reason for hiding this comment

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

Maybe this makes more sense as an array?

const endPosition = coordinates === "1,3";

const onClickGo = (type, direction) => {
console.log(`User clicked ${direction}`);

Choose a reason for hiding this comment

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

Remove console.logs in production

})
}
>
{showDirections ? "Show Location" : "Show Directions"}

Choose a reason for hiding this comment

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

I like this idea! It would be nice if it didn't show on the last image though, since there are no directions then 👀

// Function to start the game
const startGame = async () => {
if (!username.trim()) {
alert("Please enter a username.");

Choose a reason for hiding this comment

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

@@ -0,0 +1,17 @@
import styled from "styled-components";

Choose a reason for hiding this comment

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

These buttons could just be one reusable component, right?

Choose a reason for hiding this comment

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

Well done!

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