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

Unable to save LED preferences when 12mA (WS2815) is selected #3963

Open
1 task done
moinois opened this issue May 9, 2024 · 4 comments
Open
1 task done

Unable to save LED preferences when 12mA (WS2815) is selected #3963

moinois opened this issue May 9, 2024 · 4 comments
Labels
bug fixed in source This issue is unsolved in the latest release but fixed in master

Comments

@moinois
Copy link

moinois commented May 9, 2024

What happened?

Enabling the automatic brightness limiter and choosing the WS281x and then the 12mA (WS2815) option in mA/LED nothing happens when pressing Save.

To Reproduce Bug

  1. Click on Config
  2. Click on LED preferences
  3. Check the Enable automatic brightness limiter
  4. Choose WS281x in the drop down below
  5. Select 15mA (WS2815) in mA/LED dropdown
  6. Click Save

Nothing happens.

If I choose any other LED in the dropdown it is possible to save.

Expected Behavior

After klicking Save a message should appear saying LED settings saved.

Install Method

Binary from WLED.me

What version of WLED?

WLED 0.15.0-b2, build 2404100

Which microcontroller/board are you seeing the problem on?

ESP32

Relevant log/trace output

The error message I get in the console of my web browser is:
An invalid form control with name='LA0' is not focusable. <input name=​"LA0" type=​"number" min=​"1" max=​"254" oninput=​"UI()​">

Anything else?

I have the same issue using the app on iOS, but I'm assuming it uses the same web page in the end.

Code of Conduct

  • I agree to follow this project's Code of Conduct
@moinois moinois added the bug label May 9, 2024
@blazoncek
Copy link
Collaborator

This is fixed in 0_15 branch.

@blazoncek blazoncek added the fixed in source This issue is unsolved in the latest release but fixed in master label May 9, 2024
@moinois
Copy link
Author

moinois commented May 9, 2024

Okay, I'm currently using the latest beta (beta 2) in 0.15 branch so I assume it's in the pipeline then? I'll update as soon as the next beta is released to verify.

@blazoncek
Copy link
Collaborator

just grab GH CI binary from latest commit.

@dosipod
Copy link

dosipod commented May 21, 2024

Someone else reported toady it was not saving on discord and i did try on the very latest as of commit 4a7ef07 and still the behavior is funky even when selecting any led type with the limiter unchecked( workaround in video ) , this was reported a while back and I think it has came a long with the introduction of per-output limiter but not 100% sure as I did not have a proper fixture for that to test
prefrance_bug
Do not kill the messenger :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fixed in source This issue is unsolved in the latest release but fixed in master
Projects
None yet
Development

No branches or pull requests

3 participants