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

oh-gauge-card action not working, gauge not clickable #2793

Closed
dandjo opened this issue Oct 3, 2024 · 8 comments · Fixed by #2794
Closed

oh-gauge-card action not working, gauge not clickable #2793

dandjo opened this issue Oct 3, 2024 · 8 comments · Fixed by #2794
Labels
bug Something isn't working main ui Main UI

Comments

@dandjo
Copy link

dandjo commented Oct 3, 2024

The problem

I’m using oh-gauge-cards to visualize my meters. Unfortunatelty, they are not clickable anywhere when using an action.

image

Expected behavior

Being able to klick on a gauge when an action is configured.

Steps to reproduce

Configure a oh-gauge-card somewhere in the main UI.

component: oh-gauge-card
config:
  bgColor: "#ffffff"
  borderBgColor: "#cccccc"
  borderColor: "#006eff"
  borderWidth: "25"
  item: water_meter_value_day
  labelFontSize: "0"
  labelText: " "
  min: 0
  max: 0.5
  title: Water Meter Consumption
  type: semicircle
  valueFontSize: "22"
  action: analyzer
  actionAnalyzerItems:
    - water_meter_value_day

Your environment

runtimeInfo:
  version: 4.3.0
  buildString: "Build #4301"
locale: en-AT
systemInfo:
  configFolder: /etc/openhab
  userdataFolder: /var/lib/openhab
  logFolder: /var/log/openhab
  javaVersion: 17.0.12
  javaVendor: Ubuntu
  osName: Linux
  osVersion: 6.8.0-1012-raspi
  osArchitecture: aarch64
  availableProcessors: 4
  freeMemory: 168610088
  totalMemory: 662700032
  uptime: 88117
  startLevel: 100
addons:
  - automation-jsscripting
  - binding-enigma2
  - binding-http
  - binding-mielecloud
  - binding-modbus
  - binding-mqtt
  - binding-netatmo
  - binding-tado
  - misc-openhabcloud
  - persistence-influxdb
  - transformation-jsonpath
  - transformation-regex
  - ui-basic
clientInfo:
  device:
    ios: false
    android: false
    androidChrome: false
    desktop: true
    iphone: false
    ipod: false
    ipad: false
    edge: false
    ie: false
    firefox: false
    macos: false
    windows: false
    cordova: false
    phonegap: false
    electron: false
    nwjs: false
    webView: false
    webview: false
    standalone: false
    pixelRatio: 1
    prefersColorScheme: dark
  isSecureContext: false
  locationbarVisible: true
  menubarVisible: true
  navigator:
    cookieEnabled: true
    deviceMemory: N/A
    hardwareConcurrency: 8
    language: en-AT
    languages:
      - en-AT
      - en-GB
      - en-US
      - en
      - de
    onLine: true
    platform: Linux x86_64
  screen:
    width: 3440
    height: 1440
    colorDepth: 24
  support:
    touch: false
    pointerEvents: true
    observer: true
    passiveListener: true
    gestures: false
    intersectionObserver: true
  themeOptions:
    dark: dark
    filled: true
    pageTransitionAnimation: default
    bars: light
    homeNavbar: default
    homeBackground: default
    expandableCardAnimation: default
    blocklyRenderer: null
  userAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like
    Gecko) Chrome/129.0.0.0 Safari/537.36
timestamp: 2024-10-03T13:59:18.259Z
@dandjo dandjo added bug Something isn't working main ui Main UI labels Oct 3, 2024
@florian-h05
Copy link
Contributor

Thanks for noticing this! Must have been caused by my refactoring in #2781.
I will fix that regression.

florian-h05 added a commit to florian-h05/openhab-webui that referenced this issue Oct 3, 2024
Fixes openhab#2793.
Regression from openhab#2781.

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
florian-h05 added a commit that referenced this issue Oct 3, 2024
Fixes #2793.
Regression from #2781.

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
@dandjo
Copy link
Author

dandjo commented Oct 3, 2024

Thanks @florian-h05 !
By the way, same applies to oh-image-card.

@florian-h05
Copy link
Contributor

Fixed as well 👍

@dandjo
Copy link
Author

dandjo commented Oct 3, 2024

Confirmed for oh-gauge-card, the oh-image-card still broken.

component: oh-image-card
config:
  action: url
  actionUrl: http://water-meter.domotics.lan
  lazy: true
  lazyFadeIn: true
  refreshInterval: 10000
  url: http://water-meter.domotics.lan/img_tmp/alg_roi.jpg

@florian-h05
Copy link
Contributor

oh-image-card fix just merged: #2795.

@dandjo
Copy link
Author

dandjo commented Oct 3, 2024

Thanks, will test with next nightly.

@florian-h05
Copy link
Contributor

FYI seems that Jenkins already build a new snapshot containing the fix.

@dandjo
Copy link
Author

dandjo commented Oct 3, 2024

Perfect. Tested and confirmed. Thanks.

stefan-hoehn pushed a commit to stefan-hoehn/openhab-webui that referenced this issue Oct 5, 2024
)

Fixes openhab#2793.
Regression from openhab#2781.

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working main ui Main UI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants