Skip to content

Handle reactiveVal labels for shiny >=1.12#661

Merged
hadley merged 2 commits into
hadley:mainfrom
schloerke:fix-reactive-val-label
Feb 3, 2026
Merged

Handle reactiveVal labels for shiny >=1.12#661
hadley merged 2 commits into
hadley:mainfrom
schloerke:fix-reactive-val-label

Conversation

@schloerke

@schloerke schloerke commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

Detect the installed shiny version and use the appropriate internal label helper: keep using shiny:::rvalSrcrefToLabel for <1.12.0, and use shiny:::rassignSrcrefToLabel for >=1.12.0. Also extract the call srcref to a variable to avoid repeated attr calls and ensure stable label generation across shiny versions.

Fixes #659

Detect the installed shiny version and use the appropriate internal label helper: keep using shiny:::rvalSrcrefToLabel for <1.12.0, and use shiny:::rassignSrcrefToLabel for >=1.12.0, providing a defaultLabel that includes a unique id. Also extract the call srcref to a variable to avoid repeated attr calls and ensure stable label generation across shiny versions.
@schloerke schloerke force-pushed the fix-reactive-val-label branch from ffea8da to ffd7683 Compare February 3, 2026 16:36
@hadley hadley merged commit 14d42fa into hadley:main Feb 3, 2026
2 checks passed
@schloerke schloerke deleted the fix-reactive-val-label branch February 3, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

I can't build the book, I get an error at reactivity-motivation.Rmd

2 participants