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.

Sign up for GitHub

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

Jump to bottom

Alarm sensitivity is not handled correctly #51

Open
rjwills28 opened this issue Feb 29, 2024 · 2 comments
Open

Alarm sensitivity is not handled correctly #51

rjwills28 opened this issue Feb 29, 2024 · 2 comments

Comments

@rjwills28
Copy link
Collaborator

Currently cs-web-lib uses the property border_alarm_sensitive to decide whether to show a widget in the alarm state. This is inconsistent with the CSS implementation where the widget always acknowledges the alarm state and the user can optionally decide to also show this as a border.

CSS implementation for border_alarm_sensitive. Note that if false it still displays the value as red indicating it is in alarm state.
css_alarm

WebUI implementation. Note that when border_alarm_sensitive=false we get no alarm information
webui_alarm
@rjwills28 rjwills28 changed the title Alarm sensitvity is not handled correctly Alarm sensitivity is not handled correctly Feb 29, 2024
@MJGaughran
Copy link
Collaborator

What is the behaviour on Phoebus?

@rjwills28
Copy link
Collaborator Author

Interesting - different to both:
Phoebus
phoebus_alarm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@MJGaughran @rjwills28