pcdshub / lightpath

LCLS Lightpath Module
https://pcdshub.github.io/lightpath
Other
4 stars 9 forks source link

FIX: Multiple Hidden Conditions #86

Closed teddyrendahl closed 5 years ago

teddyrendahl commented 5 years ago

Description

There was a bug where multiple hidden conditions would override each other. This behavior was rather unintuitive. For a widget to be visible it should have to meet all the selected criteria not just one. This PR consolidates the hidden logic into a single filter function which is then triggered by all the buttons.

Motivation and Context

Closes #81

How Has This Been Tested?