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

Pause menu options no longer forces a song restart #14541

Open
wants to merge 2 commits into
base: experimental
Choose a base branch
from

Conversation

xenkap
Copy link

@xenkap xenkap commented Apr 26, 2024

could be better, but could be worse too (people want this !!)
if this needs a couple tweaking but otherwise good to get pulled, i'd be down to cooperate

i would've added changes that made some other restart-only settings update but it would probably break a bunch of mods + just breaking change in general even if it's like a minor thing such as timeTxt

(excuse my 250ms offset) demonstration video compressed from 88MB to 10MB for github's sake
https://github.com/ShadowMario/FNF-PsychEngine/assets/103583353/eb29c2c3-66f1-4243-b42e-20641986a6ab

could be better, but could be worse too (people want this !!)
if this needs a couple tweaking but otherwise good to get pulled, tell me!!!!

i would've added changes that made some other restart-only settings update but it would probably break a bunch of mods + just breaking change in general even if it's like a minor thing such as timeTxt
@crowplexus
Copy link
Contributor

crowplexus commented Apr 27, 2024

here's the translated text for the warning
AVISO: Nem todas as opções irão atualizar de uma vez\nalgumas apenas irão quando você reinicier a música.

@xenkap
Copy link
Author

xenkap commented Apr 27, 2024

grahhh where do i find where to put the english counterpart

@xenkap
Copy link
Author

xenkap commented Apr 27, 2024

oh wait ... oh no, do i have to write the whole english phrase in the language file oh no
nevermind i'm stupid!! learned how this works :3c

uuuh should there be an exclamation mark here ooor
@xenkap
Copy link
Author

xenkap commented Apr 27, 2024

would have named it "pause_options_warning" but uuh i see that the pause_ prefix is used for a part in the code
not like it'd conflict with anything at all but yea

Copy link

@Stefan2008Git Stefan2008Git left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be merged and accepted

@Stefan2008Git
Copy link

@xenkap You should add this code when you press BACK on note offset menu to switch state to in-game
image

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.

None yet

3 participants