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

Peki's new pull request #218

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

Peki's new pull request #218

wants to merge 29 commits into from

Conversation

akipeki
Copy link

@akipeki akipeki commented May 3, 2023

No description provided.

Copy link

@OscarSindemark OscarSindemark left a comment

Choose a reason for hiding this comment

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

Hey Peki! Your code works and is super cool with the AI images! I think a problem with it though is it sort of takes a while to load the images completely, though that may have just been a wifi issue for me! Another thing is the restart button stays but the directions move with the description text, making it so that sometimes the restart is stacked on the directions, so it looks cluttered. Otherwise, great job!

export const RestartBtn = styled(StartBtn)`
display: flex;
justify-content: center;
position: absolute;

Choose a reason for hiding this comment

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

I think because this position is absolute, it's messing with the directions in a few of the pages. It keeps moving around.

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