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

Fix Toggle Positioning and Add Win Popup in Tic Tac Toe #92

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

Conversation

TornovDutta
Copy link

Description

This PR addresses two issues in the Tic Tac Toe application: the positioning of the toggle switch and the implementation of a popup message that appears when a player wins. The CSS for the .toggle class has been updated to position it on the right side of the screen, enhancing accessibility. Additionally, a popup has been added to display the winner's name upon winning, improving user interaction. The toggle switch's position has been verified across different screen sizes, and the functionality of both the toggle and popup have been confirmed to work seamlessly.

Copy link

vercel bot commented Oct 6, 2024

@TornovDutta is attempting to deploy a commit to the iamparas0's projects Team on Vercel.

A member of the Team first needs to authorize it.

@TornovDutta
Copy link
Author

TornovDutta commented Oct 6, 2024

@iamparas0
please label it.

@TornovDutta
Copy link
Author

This is PR of #79

Copy link

vercel bot commented Oct 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tic-tac-toe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 6, 2024 11:01am

@TornovDutta
Copy link
Author

TornovDutta commented Oct 6, 2024

@iamparas0
please Label the PR

@iamparas0 iamparas0 self-requested a review October 6, 2024 17:25
@TornovDutta
Copy link
Author

@iamparas0 , is this PR okay?

@TornovDutta
Copy link
Author

@iamparas0 please label it , and the pr

@TornovDutta
Copy link
Author

@iamparas0 please check the PR

@iamparas0
Copy link
Owner

@iamparas0 please check the PR

There's a conflicting with existing code so wait I am resolving it and will merge it

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