insightsengineering / insightsengineering/teal.goshawk

Refactor the toggle slider in the package based on the comment

Open
#321 0 comments 0 reactions 0 assignees View on GitHub
core
Dominant language
R
Stars
3
Forks
3
Avg merge
2d 6h
Merged PRs (30d)
11

Description

I think that `update_toggle_slider` should update only `state` and then `state` change should update slider and numeric inputs. I think there should be only one input rendered and `if (input$toggle %% 0)` should be placed in the `renderUI` - this would avoid a need to update two inputs in the same time.

_Originally posted by @gogonzo in https://github.com/insightsengineering/teal.goshawk/pull/319#discussion_r1803012866_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.