Skip to content

feat: Switch to RNDX for rounded boxes - #61

Merged
TomDotBat merged 7 commits into
masterfrom
feat/rndx
Jul 25, 2025
Merged

feat: Switch to RNDX for rounded boxes#61
TomDotBat merged 7 commits into
masterfrom
feat/rndx

Conversation

@Lythium4848

Copy link
Copy Markdown
Collaborator

read the title

@Lythium4848
Lythium4848 requested a review from Copilot June 29, 2025 22:00
@Lythium4848

Copy link
Copy Markdown
Collaborator Author

Warning

@TomDotBat likes men

This comment was marked as outdated.

Lythium4848 and others added 3 commits June 29, 2025 23:03
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

This comment was marked as spam.

@justinkluever justinkluever left a comment

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.

Code looks fine and it works, i recommend making the variables local though, as it seems like it still improves the performance a bit on my end.

(i always test with fps_max 999 and without vsync to see minor stutters or bottlenecks)

@justinkluever justinkluever left a comment

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.

After that i would approve it i think

Comment thread lua/pixelui/drawing/cl_outlined_box.lua

@justinkluever justinkluever left a comment

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.

LGTM
you will probably still be prevented by the linting bot tho
idk how strict tom is about that

@TomDotBat

Copy link
Copy Markdown
Owner

LGTM you will probably still be prevented by the linting bot tho idk how strict tom is about that

yeah fix the linting pls

@TomDotBat
TomDotBat merged commit 2f199b4 into master Jul 25, 2025
1 check passed
@TomDotBat
TomDotBat deleted the feat/rndx branch July 25, 2025 13:02
@JackGamesFTW

Copy link
Copy Markdown

this pr is fire

@Lythium4848

Copy link
Copy Markdown
Collaborator Author

this pr is fire

thanks man i put my heart and soul into 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.

5 participants