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

[BUGFIX] A-Bot now reacts to volume on desktop #3475

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

Conversation

AlexGamingSW
Copy link

@AlexGamingSW AlexGamingSW commented Sep 28, 2024

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

Yes, #3400

Briefly describe the issue(s) fixed.

A-Bot now reacts to volume on desktop version however it's not perfect and doesn't look exactly the same as the web version, but it does look better than the previous implementation

Include any relevant screenshots or videos.

a-bot.mp4

@EliteMasterEric EliteMasterEric added the status: pending triage Awaiting review. label Oct 1, 2024
@EliteMasterEric
Copy link
Member

What happens if you apply the inverse of this to the web version instead?

@AbnormalPoof
Copy link
Collaborator

What happens if you apply the inverse of this to the web version instead?

Using this line (which I assume is the inverse):

animFrame = Math.round(animFrame / (Math.round(FlxG.sound.logToLinear(FlxG.sound.volume) * 10) / 10));
ormfroptmgtopymgportmgprot.mp4

@Hundrec Hundrec added type: minor bug Involves a minor bug or issue. size: small A small pull request with 10 or fewer changes. pr: haxe PR modifies game code. labels Jan 22, 2025
@AbnormalPoof AbnormalPoof added the topic: polish Involves minor polish to the UI or gameplay. label 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: minor bug Involves a minor bug or issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants