Skip to content

Conversation

@befaci03
Copy link

i don't know what to say, just check out commits

@befaci03 befaci03 marked this pull request as draft October 25, 2025 11:45
@befaci03 befaci03 marked this pull request as ready for review October 26, 2025 18:42
@befaci03 befaci03 marked this pull request as draft October 26, 2025 18:50
@befaci03
Copy link
Author

i have some issues with the turn status

@befaci03 befaci03 marked this pull request as ready for review October 26, 2025 20:19
@befaci03
Copy link
Author

this is ready computernewb!

Copy link

@yellows111 yellows111 left a comment

Choose a reason for hiding this comment

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

This broke in testing, failing multiple assertions.

</div>
<div class="col-md-8">
<div class="table-responsive chat-table" id="chatListDiv">
<p id="turnstatus"></p>

Choose a reason for hiding this comment

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

Explain why this has been moved to the chatbox.

Copy link
Author

Choose a reason for hiding this comment

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

I found it better being inside at the top of the chatbox, I should do some screenshots tho but I mean, so the turn status is at the top inside of the chatbox centered and keeping it at the top (I also must add a background, but I will let you think before that)

@befaci03 befaci03 requested a review from yellows111 October 27, 2025 10:32
@befaci03
Copy link
Author

everything should be fixed

@re9177
Copy link

re9177 commented Oct 27, 2025

jesus sweet finger skating christ, what in the name of fuck am i reading what is this PR holy mother of christ what the fuck did i make my eyes lay upon

@befaci03
Copy link
Author

jesus sweet finger skating christ, what in the name of fuck am i reading what is this PR holy mother of christ what the fuck did i make my eyes lay upon

what's wrong bruv 😭

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