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

Weird behavior about "Apply" button in preference page #18

Open
ghf50 opened this issue Aug 4, 2023 · 6 comments
Open

Weird behavior about "Apply" button in preference page #18

ghf50 opened this issue Aug 4, 2023 · 6 comments
Labels
bug Something isn't working

Comments

@ghf50
Copy link

ghf50 commented Aug 4, 2023

Windows 11 x64
foobar2000 v2.0 x64
foo_midi v2.9.0.0 x64

Normally, "Apply" button is greyed out when there are no changes in preference page.

If navigate to preference->Playback->Decoding->MIDI Player, the "Apply" button will available immediately.
Now, the "Apply" button will never go greyed out whatever you click it or even switch to other page until open preference page next time(not on "MIDI Player" page defaultly).

However "Apply" button still take effect.

@stuerp
Copy link
Owner

stuerp commented Aug 4, 2023

Thx. for the report. Probably a bug in the state reporting. I'll look into it.

@stuerp stuerp added the bug Something isn't working label Sep 29, 2023
@stuerp
Copy link
Owner

stuerp commented Nov 3, 2023

Fixed in v2.9.1.3.

@stuerp stuerp closed this as completed Nov 3, 2023
@ghf50
Copy link
Author

ghf50 commented Nov 3, 2023

Fixed in v2.9.1.3.

Hi. Dear stuerp. I still encounter this issue in v2.9.1.3.
v2913

@stuerp stuerp reopened this Dec 25, 2023
@stuerp
Copy link
Owner

stuerp commented May 19, 2024

Should be fixed in alpha7.

@ghf50
Copy link
Author

ghf50 commented May 20, 2024

Hi. Dear stuerp. I can confirm that this issue fixed in v2.11.0.0-alpha 7.

but change "BASS MIDI" --> "Volume" input field won't trigger "Apply" button active, and "OK" button don't save if only change "Volume" field.

other input field like "Sample rate" or "Decay time" is OK.

@stuerp
Copy link
Owner

stuerp commented May 20, 2024

You're right. Fixed in the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants