reactive questions can now show asterisks if they are required

surveydown-dev · Oct 5, 2024 · dbbe64e · dbbe64e
1 parent cd50749
commit dbbe64e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/R/ui.R b/R/ui.R
@@ -397,13 +397,9 @@ sd_question <- function(
       id = paste0("container-", id),
       `data-question-id` = id,
       class = "question-container",
-      style = sprintf("width: %s; position: relative;", width),
+      style = sprintf("width: %s;", width),
       oninput = js_interaction,
-      if (type == "matrix") {
-        output
-      } else {
-        shiny::div(output)
-      }
+      output
     )
 
     if (!is.null(shiny::getDefaultReactiveDomain())) {
@@ -869,7 +865,8 @@ sd_output <- function(id, type = NULL, display = "inline", wrapper = NULL, ...)
             id = paste0("placeholder-", id),
             `data-question-id` = id,
             class = "question-container",
-            shiny::uiOutput(id)
+            shiny::uiOutput(id),
+            shiny::tags$span(class = "hidden-asterisk", "*")
         ))
     }
 

diff --git a/inst/css/surveydown.css b/inst/css/surveydown.css
@@ -60,6 +60,7 @@ p, .form-group, .control-label {
   margin-left: auto;
   margin-right: auto;
   margin-bottom: 1rem;
+  position: relative;
 }
 
 .shiny-input-container:not(.shiny-input-container-inline) {