Fix: Have loading message adapt

Author: bnmnetp

runestone/selectquestion/selectone.py

@@ -49,7 +49,7 @@
 TEMPLATE = """
 <div class="runestone alert alert-warning sqcontainer">
 <div data-component="selectquestion" id={component_id} {selector} {points} {proficiency} {min_difficulty} {max_difficulty} {autogradable} {not_seen_ever} {primary} {AB} {toggle_options} {toggle_labels}>
-    <p>Loading ...</p>
+    <p>{message}</p>
 </div>
 </div>
 """
@@ -109,25 +109,28 @@ def run(self):
         addQuestionToDB(self)
         env = self.state.document.settings.env
         is_dynamic = env.config.html_context.get("dynamic_pages", False)
+        is_preview = env.config.html_context.get("course_id", None) == "preview"
 
         if not (bool("fromid" in self.options) ^ bool("proficiency" in self.options)):
             raise self.severe(
                 "You must specify either fromid or proficiency but not both"
             )
 
-        if is_dynamic:
-            self.options["message"] = "Loading ..."
+        if is_dynamic or is_preview:
+            self.options["message"] = "Loading a dynamic question ..."
         else:
             self.options[
                 "message"
-            ] = "The selectquestion directive only works with dynamic pages"
+            ] = "The selectquestion directive only works with Runestone Services"
 
         if "fromid" in self.options:
             self.question_bank_choices = self.options["fromid"]
             self.options[
                 "selector"
             ] = f"data-questionlist='{self.question_bank_choices}'"
-            self.options["message"] += f"\nSelecting from: {self.question_bank_choices}"
+            self.options[
+                "message"
+            ] += f"<br/>Selecting from: {self.question_bank_choices}"
         else:
             self.options["selector"] = ""