Skip to content

Implement UI skeletons#3

Merged
St0pien merged 7 commits intomainfrom
ui
Dec 23, 2024
Merged

Implement UI skeletons#3
St0pien merged 7 commits intomainfrom
ui

Conversation

@St0pien
Copy link
Member

@St0pien St0pien commented Dec 23, 2024

No description provided.

Copy link
Member Author

@St0pien St0pien left a comment

Choose a reason for hiding this comment

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

👌

fun AcceptGameScreen(
viewModel: AcceptGameViewModel = AcceptGameViewModel(),
navController: NavController,
// isRoomCreator: Boolean = false
Copy link
Member Author

Choose a reason for hiding this comment

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

Comment to remove

Player("User123", R.drawable.userimage),
Player("User456", R.drawable.userimage),
Player("User789", R.drawable.userimage))
_gameState.emit(_gameState.value.copy(players = examplePlayers))
Copy link
Member Author

Choose a reason for hiding this comment

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

Why not like this

_gameState.update {
    it.copy(players = examplePlayers)
}

Song(3, "Song 3"),
Song(4, "Song 4"),
Song(5, "Song 5"))
_gameState.emit(_gameState.value.copy(songs = exampleSongs))
Copy link
Member Author

Choose a reason for hiding this comment

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

Same as example players

@St0pien St0pien added bug Something isn't working and removed bug Something isn't working labels Dec 23, 2024
@St0pien St0pien merged commit 1a75e2f into main Dec 23, 2024
3 checks passed
@St0pien St0pien deleted the ui branch December 23, 2024 20:03
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

Comments