Skip to content

Fix Pico death spritesheet#1

Closed
doggogit wants to merge 5 commits intoFunkinCrew:mainfrom
doggogit:pico-death-fix
Closed

Fix Pico death spritesheet#1
doggogit wants to merge 5 commits intoFunkinCrew:mainfrom
doggogit:pico-death-fix

Conversation

@doggogit
Copy link
Contributor

@doggogit doggogit commented May 8, 2024

With Free Texture Packer, there is a built-in PNG optimizer, called tinyPNG.

When a spritesheet is too large and is ran through tinyPNG, it'll overcompensate and transparency will be lost, leading to a tint appearing across the entire sprite.

I re-exported the sheet using Funkin Packer (a fork of Free Texture Packer by Ne_eo), but this time, running it through only oxipng.

Even at oxipng's highest optimization flag level, no tint appears.

@chronicsilly
Copy link

Funkin Packer my beloved

@EliteMasterEric EliteMasterEric added type: minor bug Involves a minor bug or issue. status: reviewing internally Under consideration and testing. labels Jun 16, 2024
@EliteMasterEric EliteMasterEric self-assigned this Jun 16, 2024
@EliteMasterEric EliteMasterEric added the topic: polish Involves minor polish to the UI or gameplay. label Jun 16, 2024
@EliteMasterEric EliteMasterEric added status: pending triage Awaiting review. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. size: small A small pull request with 10 or fewer changes. type: minor bug Involves a minor bug or issue. and removed type: minor bug Involves a minor bug or issue. status: reviewing internally Under consideration and testing. topic: polish Involves minor polish to the UI or gameplay. status: pending triage Awaiting review. labels Jul 9, 2024
@EliteMasterEric EliteMasterEric added this to the 0.5.0 milestone Jul 13, 2024
@doggogit doggogit closed this by deleting the head repository Jul 13, 2024
@doggogit
Copy link
Contributor Author

Oh my God, I'm so stupid..

I accidentally deleted the fork by accident, Jesus Christ.

@EliteMasterEric has this been merged internally or do I need to re-open?

@doggogit
Copy link
Contributor Author

Reopened in #50. God.

@EliteMasterEric EliteMasterEric added status: resolved status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. type: minor bug Involves a minor bug or issue. and removed type: minor bug Involves a minor bug or issue. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. size: small A small pull request with 10 or fewer changes. status: resolved labels Jul 25, 2024
@EliteMasterEric EliteMasterEric added the size: small A small pull request with 10 or fewer changes. label Jul 25, 2024
@EliteMasterEric
Copy link
Member

Oh yeah this got merged internally.

Hazardous2468 pushed a commit to Hazardous2468/WITF.Assets that referenced this pull request Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: small A small pull request with 10 or fewer changes. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. type: minor bug Involves a minor bug or issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants