Merge pull request #537 from pateljannat/general

fix: misc issues

lms/www/courses/create.html

@@ -78,7 +78,7 @@
                 {{ _("Course Description") }}
             </div>
             <div class="field-description">
-                {{ _("Add a detailed description") }}
+                {{ _("Add a detailed description to provide more information about your course.") }}
             </div>
         </div>
         <div id="description" class=""></div>
@@ -109,7 +109,7 @@
                 {{ _("Tags") }}
             </div>
             <div class="field-description">
-                {{ _("Add suitable tags") }}
+                {{ _("Tags act as search keywords. They also appear on the Course Card and Course Detail page") }}
             </div>
         </div>
         <div class="tags field-input">
@@ -146,7 +146,7 @@
                 {{ _("Course Image") }}
             </div>
             <div class="field-description">
-                {{ _("Add an appropriate image") }}
+                {{ _("Image will appear on the Course Card") }}
             </div>
         </div>
         <div class="">

lms/www/courses/create.js

@@ -9,6 +9,17 @@ frappe.ready(() => {
 		create_tag(e);
 	});
 
+	$("#tags-input").focus((e) => {
+		$(e.target).keypress((e) => {
+			if (e.which == 13 || e.which == 44) {
+				create_tag(e);
+				setTimeout(() => {
+					$("#tags-input").val("");
+				}, 0);
+			}
+		});
+	});
+
 	$(document).on("click", ".btn-remove", (e) => {
 		$(e.target).parent().parent().remove();
 	});
@@ -27,14 +38,6 @@ frappe.ready(() => {
 		}
 	});
 
-	$("#tags-input").focus((e) => {
-		$(e.target).keypress((e) => {
-			if (e.which == 13) {
-				create_tag(e);
-			}
-		});
-	});
-
 	$(".btn-upload").click((e) => {
 		upload_file(e);
 	});

lms/www/courses/outline.js

@@ -49,6 +49,7 @@ const save_chapter = (e) => {
 			chapter: parent.data("chapter") || null,
 		},
 		callback: (data) => {
+			$("#chapter-modal").modal("hide");
 			frappe.show_alert({
 				message: __("Saved"),
 				indicator: "green",