Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Overlay que port #1248

Merged
merged 2 commits into from
Dec 18, 2024
Merged

Overlay que port #1248

merged 2 commits into from
Dec 18, 2024

Conversation

NPC1314
Copy link
Contributor

@NPC1314 NPC1314 commented Dec 16, 2024

About The Pull Request

ports
Vanderlin-Tales-Of-Wine#69

lets see if it breaks anything I didnt notice wohoo

Pre-Merge Checklist

  • You tested this on a local server.
  • This code did not runtime during testing.
  • You documented all of your changes.

@NPC1314 NPC1314 added the Scheduled for testmerge This PR should be testmerged label Dec 16, 2024
@NameChangeStonekeep
Copy link

this is just a slightly modified tg inventory slot UI... yucky..

@NPC1314
Copy link
Contributor Author

NPC1314 commented Dec 17, 2024

this is just a slightly modified tg inventory slot UI... yucky..

Its the removal of a entire resource heavy overlay system that improves perfomance noticeably.
That the inventory ui lost the overlay framed is just a unfortunate side effect, if borlop re-adds it we can get it back, if not theres not a lot we can do.

@NPC1314 NPC1314 merged commit c25b203 into Darkrp-community:main Dec 18, 2024
7 checks passed
@NPC1314 NPC1314 removed the Scheduled for testmerge This PR should be testmerged label Dec 18, 2024
@NameChangeStonekeep
Copy link

this is just a slightly modified tg inventory slot UI... yucky..

Its the removal of a entire resource heavy overlay system that improves perfomance noticeably. That the inventory ui lost the overlay framed is just a unfortunate side effect, if borlop re-adds it we can get it back, if not theres not a lot we can do.

but that's what gave it le soul ;c

@NPC1314 NPC1314 deleted the overlays branch January 6, 2025 13:39
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