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

fixes #1513 #2513

Merged
merged 1 commit into from
Nov 16, 2023
Merged

fixes #1513 #2513

merged 1 commit into from
Nov 16, 2023

Conversation

jp112sdl
Copy link
Contributor

Description

After creating the dimmer controls, a refresh method sets the correct css style (by setting certain classes) for the ON and OFF buttons. This is done with a timeout of 1 second and leads to a strange optical behaviour as described in the related issue.

Setting this timeout to zero fixes it.

Related Issue

#1513

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Alternate Designs

Possible Drawbacks

Only tested with HM-LC-Dim1TPBU-FM and HM-LC-Dim1T-FM

Verification Process

Release Notes

Contributing checklist

  • My code follows the code style of this project.
  • I have read the CONTRIBUTING and LICENSE document.
  • I fully agree to distribute my changes under Apache 2.0 license.

@jens-maus jens-maus merged commit d250419 into jens-maus:master Nov 16, 2023
4 checks passed
@jens-maus jens-maus added 🐛 bug-report Something isn't working 🏷️ WebUI This refs the WebUI component labels Nov 16, 2023
@jens-maus jens-maus added this to the next release milestone Nov 16, 2023
jp112sdl added a commit to jp112sdl/RaspberryMatic that referenced this pull request Nov 18, 2023
…into dev_patch0182_fix1513

* 'master' of https://github.com/jens-maus/RaspberryMatic:
  snapshot bump [3.73.9.20231118-bf6c11c]
  Add port 2110 for homeputer Exec-Engine to HA-addon config (jens-maus#2515, @tomquist)
  updated OCCU to 3.73.9-1 and adapted WebUI patches.
  explicitly disable the bcm283x serial driver support in the rpi versions of u-boot so that they won't interfere with our rpi-rf-mode dtbo. This should allow to remove the stdout-path workaround to prevent u-boot from crashing in case the rpi-rf-mod.dtbo is enabled in config.txt. (cf. home-assistant/operating-system#2919)
  snapshot bump [3.73.6.20231117-d250419]
  added new WebUI 0182-WebUI-Fix-DimmerControl-RemoveButtonRefreshDelay patch which fixes unwanted delay in displaying the correct dimmer status (jens-maus#1513, jens-maus#2513, @jp112sdl)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug-report Something isn't working 🏷️ WebUI This refs the WebUI component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HMW-LC-Dim1L-DR Dimmaktor: Aktueller Schaltzustand wird verzögert angezeigt/nachkorrigiert
2 participants