From 5f1e05298c7f18a1fa48f19cab456fffab0d51df Mon Sep 17 00:00:00 2001
From: Zameel Hassan <43750093+zameel7@users.noreply.github.com>
Date: Fri, 18 Oct 2024 10:55:47 +0530
Subject: [PATCH] feat: Add option to turn on or off fullscreen button in SCORM
 content (#83)

* feat: Add option to turn on or off fullscreen button in SCORM content
---
 openedxscorm/scormxblock.py               |  9 +++++++++
 openedxscorm/static/html/scormxblock.html |  5 +++--
 openedxscorm/static/html/studio.html      | 11 +++++++++++
 openedxscorm/static/js/src/studio.js      |  2 ++
 4 files changed, 25 insertions(+), 2 deletions(-)

diff --git a/openedxscorm/scormxblock.py b/openedxscorm/scormxblock.py
index 596a145..047028a 100644
--- a/openedxscorm/scormxblock.py
+++ b/openedxscorm/scormxblock.py
@@ -164,6 +164,12 @@ def scorm_storage(xblock):
         default=False,
         scope=Scope.settings,
     )
+    enable_fullscreen_button = Boolean(
+        display_name=_("Show Fullscreen Button"),
+        help=_("Select True to show fullscreen button in the SCORM content"),
+        default=True,
+        scope=Scope.settings,
+    )
 
     navigation_menu = String(scope=Scope.settings, default="")
 
@@ -278,6 +284,7 @@ def studio_view(self, context=None):
             "field_height": self.fields["height"],
             "field_popup_on_launch": self.fields["popup_on_launch"],
             "field_enable_navigation_menu": self.fields["enable_navigation_menu"],
+            "field_enable_fullscreen_button": self.fields["enable_fullscreen_button"],
             "field_navigation_menu_width": self.fields["navigation_menu_width"],
             "popup_on_launch": self.fields["popup_on_launch"],
             "scorm_xblock": self,
@@ -303,6 +310,7 @@ def studio_submit(self, request, _suffix):
         self.height = parse_int(request.params["height"], None)
         self.has_score = request.params["has_score"] == "1"
         self.enable_navigation_menu = request.params["enable_navigation_menu"] == "1"
+        self.enable_fullscreen_button = request.params["enable_fullscreen_button"] == "1"
         self.navigation_menu_width = parse_int(
             request.params["navigation_menu_width"], None
         )
@@ -344,6 +352,7 @@ def popup_window(self, request, _suffix):
                 "navigation_menu": self.navigation_menu,
                 "navigation_menu_width": self.navigation_menu_width,
                 "enable_navigation_menu": self.enable_navigation_menu,
+                "enable_fullscreen_button": self.enable_fullscreen_button,
             },
         )
         return Response(body=rendered)
diff --git a/openedxscorm/static/html/scormxblock.html b/openedxscorm/static/html/scormxblock.html
index 33622b1..0032b6d 100644
--- a/openedxscorm/static/html/scormxblock.html
+++ b/openedxscorm/static/html/scormxblock.html
@@ -9,12 +9,13 @@
   {% endif %}
 
   <div class="scorm-content">
-
+            {% if scorm_xblock.enable_fullscreen_button %}
             <div class="fullscreen-controls">
                 <button class="enter-fullscreen">{% trans "Fullscreen" %}</button>
                 <button class="exit-fullscreen">{% trans "Exit fullscreen" %}</button>
             </div>
-            
+            {% endif %}
+
             <div class="scorm-panel">
 
                 {% if scorm_xblock.enable_navigation_menu and not scorm_xblock.popup_on_launch%}
diff --git a/openedxscorm/static/html/studio.html b/openedxscorm/static/html/studio.html
index dd0b0af..50daa14 100644
--- a/openedxscorm/static/html/studio.html
+++ b/openedxscorm/static/html/studio.html
@@ -43,6 +43,17 @@
             <span class="tip setting-help">{% trans field_enable_navigation_menu.help %}</span>
         </li>
 
+        <li class="field comp-setting-entry is-set">
+            <div class="wrapper-comp-setting">
+                <label class="label setting-label" for="enable_fullscreen_button">{% trans field_enable_fullscreen_button.display_name %}</label>
+                <select name="enable_fullscreen_button" id="enable_fullscreen_button">
+                    <option value="1" {% if scorm_xblock.enable_fullscreen_button %}selected{% endif %}>{% trans "True" %}</option>
+                    <option value="0" {% if not scorm_xblock.enable_fullscreen_button %}selected{% endif %}>{% trans "False" %}</option>
+                </select>
+            </div>
+            <span class="tip setting-help">{% trans field_enable_fullscreen_button.help %}</span>
+        </li>
+
         <li class="field comp-setting-entry is-set">
             <div class="wrapper-comp-setting">
                 <label class="label setting-label" for="weight">{% trans field_weight.display_name %}</label>
diff --git a/openedxscorm/static/js/src/studio.js b/openedxscorm/static/js/src/studio.js
index 5aefd0b..470deb7 100644
--- a/openedxscorm/static/js/src/studio.js
+++ b/openedxscorm/static/js/src/studio.js
@@ -8,6 +8,7 @@ function ScormStudioXBlock(runtime, element) {
         var display_name = $(element).find('input[name=display_name]').val();
         var has_score = $(element).find('select[name=has_score]').val();
         var enable_navigation_menu = $(element).find('select[name=enable_navigation_menu]').val();
+        var enable_fullscreen_button = $(element).find('select[name=enable_fullscreen_button]').val();
         var weight = $(element).find('input[name=weight]').val();
         var width = $(element).find('input[name=width]').val();
         var height = $(element).find('input[name=height]').val();
@@ -18,6 +19,7 @@ function ScormStudioXBlock(runtime, element) {
         form_data.append('display_name', display_name);
         form_data.append('has_score', has_score);
         form_data.append('enable_navigation_menu', enable_navigation_menu);
+        form_data.append('enable_fullscreen_button', enable_fullscreen_button);
         form_data.append('weight', weight);
         form_data.append('width', width);
         form_data.append('height', height);