Skip to content

a lot of bugfixes - #414

Open
sad-username-231 wants to merge 110 commits into
IntQuant:masterfrom
sad-username-231:master
Open

a lot of bugfixes#414
sad-username-231 wants to merge 110 commits into
IntQuant:masterfrom
sad-username-231:master

Conversation

@sad-username-231

Copy link
Copy Markdown
Contributor

delete button functions as delete button, ctrl+ delete implemented, old features with saving chat when game is closed + its disablable, chat window is centered, russian or actually any symbols thats 2-3-4 bytes works properly now, chat color setting working wrong fixed and a few minor bug fixes.
no changing color&nickname in chat history yet sorry. will add it in next pull with some quality additions

@sad-username-231

Copy link
Copy Markdown
Contributor Author

bruh ive found critical bug

@sad-username-231
sad-username-231 marked this pull request as draft July 22, 2025 00:51
@sad-username-231
sad-username-231 marked this pull request as ready for review July 24, 2025 02:00
@sad-username-231

Copy link
Copy Markdown
Contributor Author

its all good

@sad-username-231
sad-username-231 marked this pull request as draft July 26, 2025 20:12
@sad-username-231
sad-username-231 marked this pull request as ready for review July 29, 2025 17:20
@sad-username-231

Copy link
Copy Markdown
Contributor Author

reverted run separator msg


local in_camera_ref

local appdata = os.getenv("APPDATA")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Consider saving files on local proxy directory since APPDATA works ONLY on windows and returns nil on macos and linux

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

no it would return the appdata fine, since noiter is only windows so macos/linux is proton/wine

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

oh, i did not account that, i forgot ye my bad

@Multirious

Multirious commented Jun 7, 2026

Copy link
Copy Markdown
Collaborator

Sorry, you may have to rebase your changes since we've recently cleaned up our master branch. Also, are we planning to merge this? Seems to linger for awhile now. And we would be appreciated if the commits were clean up and describe what were changed in their appropriate scope since it seems to be really difficult to review.

@sad-username-231

Copy link
Copy Markdown
Contributor Author

hi im kinda new to contributing so please tell me what exactly are my errors in commits descriptions (exept their absense) and ill be better in the future. i tried to name all my commits the way so their names shortly overview their content.
maybe this will help: in a few words only first 2 commits actually add something and they have descriptions that you can use to review. all further commits are small fixes for bugs i discovered later and some small changes that i failed to correctly implement and reverted. also this pr is based on my older pr that i canceled for some reason.
p.s: i also wanna know if it's gonna get merged. i'll redo when i'll know the time has come. i may not see your comments here for a long time so you can contact me on discord to force me to fix it. im @Dragonborn422

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.

4 participants