💻 new customize adventure page

app.py

@@ -257,6 +257,9 @@ def load_customized_adventures(level, customizations, into_adventures):
                 else:
                     db_row['content'] = safe_format(db_row['content'],
                                                     **hedy_content.KEYWORDS.get(g.keyword_lang))
+                if 'solution_example' in db_row:
+                    db_row['solution_example'] = safe_format(db_row['solution_example'],
+                                                             **hedy_content.KEYWORDS.get(g.keyword_lang))
             except Exception:
                 # We don't want teacher being able to break the student UI -> pass this adventure
                 pass

messages.pot

@@ -1589,6 +1589,9 @@ msgstr ""
 msgid "social_media"
 msgstr ""
 
+msgid "solution_example"
+msgstr ""
+
 msgid "something_went_wrong_keyword_parsing"
 msgstr ""
 
@@ -1754,6 +1757,9 @@ msgstr ""
 msgid "this_turns_in_assignment"
 msgstr ""
 
+msgid "tips"
+msgstr ""
+
 msgid "title"
 msgstr ""
 

static/js/adventure.ts

@@ -13,13 +13,15 @@ declare let window: CustomWindow;
 
 export interface InitializeCustomizeAdventurePage {
     readonly page: 'customize-adventure';
+    readonly level: number;
 }
 
 let $editor: ClassicEditor;
 let keywordHasAlert: Map<string, boolean> = new Map()
 
 export async function initializeCustomAdventurePage(_options: InitializeCustomizeAdventurePage) {
-    const editorContainer = document.querySelector('#adventure_editor') as HTMLElement;
+    const editorContainer = document.querySelector('#adventure-editor') as HTMLElement;
+    const editorSolutionExampleContainer = document.querySelector('#adventure-solution-editor') as HTMLElement;
     // Initialize the editor with the default language
     let lang =  document.querySelector('#languages_dropdown> .option.selected')!.getAttribute('data-value') as string
     const TRADUCTIONS = convert(TRADUCTION_IMPORT) as Map<string, Map<string,string>>;    
@@ -28,6 +30,7 @@ export async function initializeCustomAdventurePage(_options: InitializeCustomiz
 
     if (editorContainer) {
         await initializeEditor(lang, editorContainer);
+        await initializeEditor(lang, editorSolutionExampleContainer, true);
         showWarningIfMultipleKeywords(TRADUCTION)
         $editor.model.document.on('change:data', () => {
             showWarningIfMultipleKeywords(TRADUCTION)
@@ -107,12 +110,8 @@ function findCoincidences(name: string, TRADUCTION: Map<string, string>) {
     return coincidences;
 }
 
-function initializeEditor(language: string, editorContainer: HTMLElement): Promise<void> {
+function initializeEditor(language: string, editorContainer: HTMLElement, solutionExample=false): Promise<void> {
     return new Promise((resolve, reject) => {
-        if ($editor) {
-            $editor.destroy();
-        }
-
         ClassicEditor
             .create(editorContainer, {
                 codeBlock: {
@@ -123,9 +122,13 @@ function initializeEditor(language: string, editorContainer: HTMLElement): Promi
                 language,
             })
             .then(editor => {
-                window.ckEditor = editor;
-                $editor = editor;
-                $editor.model.document.on("change:data", e => autoSave("customize_adventure", e))
+                if (solutionExample) {
+                    window.ckSolutionEditor = editor;
+                } else {
+                    window.ckEditor = editor;
+                    $editor = editor;
+                }
+                editor.model.document.on("change:data", e => autoSave("customize_adventure", e))
                 resolve();
             })
             .catch(error => {

static/js/appbundle.js

@@ -103888,7 +103888,8 @@ def note_with_error(value, err):
   var $editor;
   var keywordHasAlert = new Map();
   async function initializeCustomAdventurePage(_options) {
-    const editorContainer = document.querySelector("#adventure_editor");
+    const editorContainer = document.querySelector("#adventure-editor");
+    const editorSolutionExampleContainer = document.querySelector("#adventure-solution-editor");
     let lang = document.querySelector("#languages_dropdown> .option.selected").getAttribute("data-value");
     const TRADUCTIONS = convert(highlighting_trad_default);
     if (!TRADUCTIONS.has(lang)) {
@@ -103897,6 +103898,7 @@ def note_with_error(value, err):
     let TRADUCTION3 = TRADUCTIONS.get(lang);
     if (editorContainer) {
       await initializeEditor(lang, editorContainer);
+      await initializeEditor(lang, editorSolutionExampleContainer, true);
       showWarningIfMultipleKeywords(TRADUCTION3);
       $editor.model.document.on("change:data", () => {
         showWarningIfMultipleKeywords(TRADUCTION3);
@@ -103970,11 +103972,8 @@ def note_with_error(value, err):
     }
     return coincidences;
   }
-  function initializeEditor(language2, editorContainer) {
+  function initializeEditor(language2, editorContainer, solutionExample = false) {
     return new Promise((resolve2, reject) => {
-      if ($editor) {
-        $editor.destroy();
-      }
       import_ckeditor.default.create(editorContainer, {
         codeBlock: {
           languages: [
@@ -103983,9 +103982,13 @@ def note_with_error(value, err):
         },
         language: language2
       }).then((editor) => {
-        window.ckEditor = editor;
-        $editor = editor;
-        $editor.model.document.on("change:data", (e) => autoSave("customize_adventure", e));
+        if (solutionExample) {
+          window.ckSolutionEditor = editor;
+        } else {
+          window.ckEditor = editor;
+          $editor = editor;
+        }
+        editor.model.document.on("change:data", (e) => autoSave("customize_adventure", e));
         resolve2();
       }).catch((error2) => {
         console.error(error2);
@@ -118472,18 +118475,7 @@ def note_with_error(value, err):
       });
     });
   }
-  function update_db_adventure(adventure_id) {
-    const adventure_name = $("#custom_adventure_name").val();
-    let classes = [];
-    let levels = [];
-    document.querySelectorAll("#levels_dropdown > .option.selected").forEach((el3) => {
-      levels.push(el3.getAttribute("data-value"));
-    });
-    document.querySelectorAll("#classes_dropdown > .option.selected").forEach((el3) => {
-      classes.push(el3.getAttribute("data-value"));
-    });
-    const language2 = document.querySelector("#languages_dropdown> .option.selected").getAttribute("data-value");
-    const content2 = import_dompurify2.default.sanitize(window.ckEditor.getData());
+  function get_formatted_content(content2, levels, language2) {
     const parser19 = new DOMParser();
     const html = parser19.parseFromString(content2, "text/html");
     const minLevel = Math.min(...levels.map((el3) => Number(el3)));
@@ -118514,6 +118506,23 @@ def note_with_error(value, err):
       }
     }
     const formatted_content = html.getElementsByTagName("body")[0].outerHTML.replace(/<br>/g, "\n");
+    return formatted_content;
+  }
+  function update_db_adventure(adventure_id) {
+    const adventure_name = $("#custom_adventure_name").val();
+    let classes = [];
+    let levels = [];
+    document.querySelectorAll("#levels_dropdown > .option.selected").forEach((el3) => {
+      levels.push(el3.getAttribute("data-value"));
+    });
+    document.querySelectorAll("#classes_dropdown > .option.selected").forEach((el3) => {
+      classes.push(el3.getAttribute("data-value"));
+    });
+    const language2 = document.querySelector("#languages_dropdown> .option.selected").getAttribute("data-value");
+    const content2 = import_dompurify2.default.sanitize(window.ckEditor.getData());
+    const solutionExampleCode = import_dompurify2.default.sanitize(window.ckSolutionEditor.getData());
+    const formatted_content = get_formatted_content(content2, levels, language2);
+    const formatted_solution_code = get_formatted_content(solutionExampleCode, levels, language2);
     const agree_public = $("#agree_public").prop("checked");
     $.ajax({
       type: "POST",
@@ -118523,6 +118532,7 @@ def note_with_error(value, err):
         name: adventure_name,
         content: content2,
         formatted_content,
+        formatted_solution_code,
         public: agree_public,
         language: language2,
         classes,
@@ -121432,16 +121442,20 @@ def note_with_error(value, err):
 
   // static/js/initialize.ts
   function initialize(options) {
-    var _a3;
+    var _a3, _b;
     setClientMessageLanguage(options.lang);
+    let level3 = options.level;
+    if (!level3 && ((_a3 = options.javascriptPageOptions) == null ? void 0 : _a3.page) == "customize-adventure") {
+      level3 = options.javascriptPageOptions.level;
+    }
     initializeApp({
-      level: options.level,
+      level: level3,
       keywordLanguage: options.keyword_language,
       staticRoot: options.staticRoot
     });
     initializeFormSubmits();
     initializeTutorial();
-    switch ((_a3 = options.javascriptPageOptions) == null ? void 0 : _a3.page) {
+    switch ((_b = options.javascriptPageOptions) == null ? void 0 : _b.page) {
       case "code":
         initializeCodePage(options.javascriptPageOptions);
         break;

static/js/custom-window.ts

@@ -7,4 +7,5 @@ import ClassicEditor from "@ckeditor/ckeditor5-build-classic";
 */
 export interface CustomWindow extends Window {
     ckEditor: ClassicEditor;
+    ckSolutionEditor: ClassicEditor;
 }

static/js/initialize.ts

@@ -66,8 +66,14 @@ type InitializePageOptions =
 export function initialize(options: InitializeOptions) {
   setClientMessageLanguage(options.lang);
 
+  let level = options.level;
+
+  if (!level && options.javascriptPageOptions?.page == "customize-adventure") {
+    level = options.javascriptPageOptions.level
+  }
+
   initializeApp({
-    level: options.level,
+    level: level,
     keywordLanguage: options.keyword_language,
     staticRoot: options.staticRoot,
   });

static/js/teachers.ts

@@ -215,24 +215,7 @@ export function remove_student(class_id: string, student_id: string, prompt: str
   });
 }
 
-function update_db_adventure(adventure_id: string) {
-  // Todo TB: It would be nice if we improve this with the formToJSON() function once #3077 is merged
-  const adventure_name = $('#custom_adventure_name').val();
-  let classes: string[] = [];
-  let levels: string[] = []
-
-  document.querySelectorAll('#levels_dropdown > .option.selected').forEach((el) => {
-    levels.push(el.getAttribute("data-value") as string)
-  })
-
-  document.querySelectorAll('#classes_dropdown > .option.selected').forEach((el) => {
-    classes.push(el.getAttribute("data-value") as string)
-  })
-
-  const language = document.querySelector('#languages_dropdown> .option.selected')!.getAttribute('data-value') as string
-
-  const content = DOMPurify.sanitize(window.ckEditor.getData());
-
+function get_formatted_content(content: string, levels: string[], language: string) {
   const parser = new DOMParser();
   const html = parser.parseFromString(content, 'text/html');
   const minLevel = Math.min(...levels.map((el) => Number(el)));
@@ -268,6 +251,30 @@ function update_db_adventure(adventure_id: string) {
   }
   // We have to replace <br> for newlines, because the serializer swithces them around
   const formatted_content = html.getElementsByTagName('body')[0].outerHTML.replace(/<br>/g, '\n');
+  return formatted_content
+}
+
+function update_db_adventure(adventure_id: string) {
+  // Todo TB: It would be nice if we improve this with the formToJSON() function once #3077 is merged
+  const adventure_name = $('#custom_adventure_name').val();
+  let classes: string[] = [];
+  let levels: string[] = []
+
+  document.querySelectorAll('#levels_dropdown > .option.selected').forEach((el) => {
+    levels.push(el.getAttribute("data-value") as string)
+  })
+
+  document.querySelectorAll('#classes_dropdown > .option.selected').forEach((el) => {
+    classes.push(el.getAttribute("data-value") as string)
+  })
+
+  const language = document.querySelector('#languages_dropdown> .option.selected')!.getAttribute('data-value') as string
+
+  const content = DOMPurify.sanitize(window.ckEditor.getData());
+  const solutionExampleCode = DOMPurify.sanitize(window.ckSolutionEditor.getData());
+
+  const formatted_content = get_formatted_content(content, levels, language);
+  const formatted_solution_code = get_formatted_content(solutionExampleCode, levels, language);
   const agree_public = $('#agree_public').prop('checked');
 
   $.ajax({
@@ -278,6 +285,7 @@ function update_db_adventure(adventure_id: string) {
       name: adventure_name,
       content: content,
       formatted_content: formatted_content,
+      formatted_solution_code: formatted_solution_code,
       public: agree_public,
       language,
       classes,