DMX Output support exclude message incorrectly displayed on builds *with* support

[netmindz]

Builds display message saying no DMX output support for builds which do contain this feature

Summary by CodeRabbit

• Refactor
  • Improved the display handling for DMX output settings in the Sync configuration page, ensuring proper visibility controls for the DMX output section.

[coderabbitai]

Walkthrough

Function and DOM element renamed from "dmxOnOff2"/"hideNoDMX" to "dmxOnOffOutput"/"hideNoDMXOutput" in the SYNC settings page. Updated both the JavaScript function definition and its call site in the XML configuration.

Changes

Cohort / File(s)	Summary
DMX Output Function Rename wled00/data/settings_sync.htm, wled00/xml.cpp	Renamed JavaScript function from hideNoDMX() to hideNoDMXOutput() and updated the associated DOM container id from "dmxOnOff2" to "dmxOnOffOutput" to improve clarity regarding DMX output visibility handling.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested reviewers

• softhack007

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the bug being fixed: a misleading message about DMX output support appearing on builds that actually have support.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}