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

User-Friendly Sprinting #889

Merged

Conversation

Mnemotechnician
Copy link
Contributor

Description

"Walking by default" is now implemented as a client-side toggle in the settings. Also, fixed walking alerts being shitcode and not displaying the correct walking/running state.

One issue still persists: if you press the "toggle walk" button while typing in the chat, it still resets the walking state to default. I don't know what causes it, probably the input field leaking button-up input events instead of marking them as handled.

Media

8mb.video-Ngx-S6se6IX1.mp4

Changelog

🆑

  • add: You can now choose whether you want to walk or run by default in the settings.

@github-actions github-actions bot added Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: UI Changes any XAML files labels Sep 8, 2024
@SimpleStation14 SimpleStation14 changed the title User-friendly sprinting User-Friendly Sprinting Sep 8, 2024
@github-actions github-actions bot added the Status: Merge Conflict FIX YOUR PR AAAGH label Sep 8, 2024
Copy link
Contributor

github-actions bot commented Sep 8, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Status: Merge Conflict FIX YOUR PR AAAGH label Sep 8, 2024
@FoxxoTrystan FoxxoTrystan added Size: 3-Medium For medium issues/PRs Priority: 3-Medium Needs to be resolved at some point labels Sep 14, 2024
@VMSolidus VMSolidus merged commit c49ce78 into Simple-Station:master Sep 14, 2024
11 checks passed
SimpleStation14 added a commit that referenced this pull request Sep 14, 2024
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: UI Changes any XAML files Priority: 3-Medium Needs to be resolved at some point Size: 3-Medium For medium issues/PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants