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

[ENHANCEMENT]: Note Splashes Centering #3433

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

JVNpixels
Copy link
Contributor

@JVNpixels JVNpixels commented Sep 22, 2024

Does this PR close any issues? If so, link them below.

#3105
#3427
#3429

Briefly describe the issue(s) fixed.

The note splashes width offset gets changed from 0.3 to 0.2

Include any relevant screenshots or videos.

Before:

image
image
image
image

After:

image
image
image
image

@AbnormalPoof
Copy link
Collaborator

I recommend adding Before/After screenshots, this PR might be a bit confusing for a couple of the reviewers.

@JVNpixels
Copy link
Contributor Author

I am busy at the moment but will do once I am back!

@JVNpixels
Copy link
Contributor Author

JVNpixels commented Sep 22, 2024

After images soon, I am finished with my other work, takes a ton of attempts to get the perfect screenshot of the exact frame of the arrow!

@JVNpixels JVNpixels changed the base branch from main to develop September 22, 2024 19:47
@JVNpixels JVNpixels changed the base branch from develop to main September 22, 2024 19:48
@CEliuxJV
Copy link

I mentioned this in #3105 as well, I hope this gets accepted, just that after the change, the center of the splashes is still not perfectly centered, if I knew how to change it it would help with this.

@EliteMasterEric EliteMasterEric added the status: pending triage Awaiting review. label Sep 25, 2024
@EliteMasterEric EliteMasterEric changed the base branch from main to develop September 25, 2024 12:56
@Hundrec Hundrec added the type: enhancement Involves an enhancement or new feature. label Jan 22, 2025
@Hundrec
Copy link
Collaborator

Hundrec commented Jan 22, 2025

I like this one a lot

@Hundrec Hundrec added size: small A small pull request with 10 or fewer changes. pr: haxe PR modifies game code. topic: polish Involves minor polish to the UI or gameplay. labels Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: haxe PR modifies game code. size: small A small pull request with 10 or fewer changes. status: pending triage Awaiting review. topic: polish Involves minor polish to the UI or gameplay. type: enhancement Involves an enhancement or new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants