Skip to content

Commit

Permalink
🐛 Opacity slider needs to depend on toggle
Browse files Browse the repository at this point in the history
...instead of current disabled state.
  • Loading branch information
Wiebke committed May 14, 2024
1 parent e0074bb commit ff9f83e
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions callbacks/control_bar.py
Original file line number Diff line number Diff line change
Expand Up @@ -872,25 +872,25 @@ def refresh_data_client(refresh_tiled):
Input("show-result-overlay-toggle", "checked"),
Input("seg-results-train-store", "data"),
Input("seg-results-inference-store", "data"),
State("seg-result-opacity-slider", "disabled"),
)
def update_result_controls(
toggle, seg_result_train, seg_result_inference, slider_disabled
):
checked = False
disable_toggle = True
disable_slider = True
def update_result_controls(toggle, seg_result_train, seg_result_inference):
# Disable opacity slider if result overlay is unchecked
if ctx.triggered_id == "show-result-overlay-toggle":
checked = no_update
# Must have been enabled to be source of trigger
disable_toggle = no_update
disable_slider = not slider_disabled
# Disable slider if toggle is unchecked
disable_slider = not toggle
# Trigger is a change in either a train or inference result
else:
if seg_result_train or seg_result_inference:
checked = False
checked = no_update
disable_toggle = False
disable_slider = False
else:
checked = False
disable_toggle = True
disable_slider = True
return (
checked,
disable_toggle,
Expand Down

0 comments on commit ff9f83e

Please sign in to comment.