Skip to content

Add fault icons for org.knx (KNX smoke sensor capabilities) - #410

Closed
LaurensVanAcker wants to merge 1 commit into
athombv:masterfrom
LaurensVanAcker:add-org.knx-fault-icons
Closed

Add fault icons for org.knx (KNX smoke sensor capabilities)#410
LaurensVanAcker wants to merge 1 commit into
athombv:masterfrom
LaurensVanAcker:add-org.knx-fault-icons

Conversation

@LaurensVanAcker

Copy link
Copy Markdown

App ID: org.knx

These icons are needed for athombv/org.knx#85, which extends the KNX smoke sensor with the full set of fault and signalling channels exposed by Gira smoke alarm objects.

Adds 4 monochrome 64x64 SVGs under org.knx/:

  • knx_failure_smoke.svg — smoke sensor failure
  • knx_failure_temperature.svg — temperature sensor failure
  • knx_failure_230v.svg — 230V power failure
  • knx_acknowledge_fault.svg — reset/acknowledge button

The 5th new capability in that PR (knx_alarm_signal, the siren control) reuses the existing homey capability Icons/alarm_siren.svg — no new file needed.

These are first-pass drafts in the existing flat monochrome style — happy to iterate if the design team prefers a different direction.

cc @wlcrs

For PR athombv/org.knx#85, which extends the KNX smoke sensor with
extra fault and signalling channels exposed by Gira smoke alarms.

Adds 4 monochrome 64x64 SVGs for the new capabilities:
- knx_failure_smoke      (smoke sensor failure)
- knx_failure_temperature (temperature sensor failure)
- knx_failure_230v        (230V power failure)
- knx_acknowledge_fault   (reset/acknowledge button)

The alarm_signal (siren) capability reuses the existing
"homey capability Icons/alarm_siren.svg" — no new file needed.
@LaurensVanAcker

Copy link
Copy Markdown
Author

Closing — I misread the icon workflow. The icons belong inside the app at assets/capabilities/ referenced via the "icon" prop on each capability JSON, not in this repo. They are now in athombv/org.knx#85. Sorry for the noise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant