Skip to content

Conversation

@DylanDevelops
Copy link
Owner

This pull request includes updates to the GameResultPage and PlayPage components to handle the introduction of a new game mode and improve user interface feedback.

Changes related to game mode handling and UI updates:

  • app/(gameplay)/results/page.tsx: Added a conditional check to exclude certain UI elements when the game mode is not "images" or "daily_challenge". This is a temporary change until the text mode is introduced. [1] [2]
  • app/(main)/play/page.tsx: Updated the "Play Text" button to be non-clickable and display "COMING SOON" to indicate that the text mode is not yet available.

IMG_0527

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

app/(gameplay)/results/page.tsx:107

  • The onClick handler inside this div is checking for gamemode values that will never be true given the outer negated condition. Consider removing the inner if condition (and the onClick handler altogether) to clearly indicate that text mode is disabled.
if(gamemode == "images" || gamemode == "daily_challenge") {

@divinewton divinewton merged commit 1d96dd1 into main Apr 14, 2025
4 checks passed
@divinewton divinewton deleted the play-page-coming-soon branch April 14, 2025 15:49
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