From bb0ae30394fc57ae0c44b20e511172c6163367f4 Mon Sep 17 00:00:00 2001
From: erseco <info@ernesto.es>
Date: Sat, 9 May 2026 13:29:38 +0100
Subject: [PATCH 1/2] fix: show Edit button in popup/open/new modes; remove
 FRAME mode (#43)

- Render the action bar (eXeLearning Edit button) in the workaround
  view used by Display = In pop-up, Open and New window. The button
  was previously only available in Embed mode, leaving teachers no
  way to edit the package from any of these display modes.
- Remove the legacy In frame (HTML4 frameset) display option per
  reviewer feedback. The mode duplicated Embed (which already uses an
  iframe) and made the editor modal and fullscreen control unusable.
  Existing activities saved with display=FRAME are migrated to EMBED
  on upgrade and the orphan framesize admin setting is dropped.
---
 CHANGELOG.md       |  5 +++
 db/upgrade.php     | 14 ++++++--
 lang/ca/exeweb.php |  3 --
 lang/en/exeweb.php |  3 --
 lang/es/exeweb.php |  3 --
 lang/eu/exeweb.php |  3 --
 lang/gl/exeweb.php |  3 --
 lib.php            |  2 +-
 locallib.php       | 86 ++--------------------------------------------
 mod_form.php       |  3 +-
 settings.php       |  4 ---
 view.php           |  5 ++-
 12 files changed, 24 insertions(+), 110 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index e98321d..8ab50ef 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,10 @@
 # CHANGELOG
 
+## Unreleased
+
+- Show the eXeLearning **Edit** button on the activity view when **Display** is set to *In pop-up*, *Open* or *New window* (issue #43). Previously the button only appeared for *Embed* activities, leaving teachers no way to edit the package from those modes.
+- Remove the legacy *In frame* (HTML4 frameset) display option. The mode duplicated *Embed* (which already uses an `<iframe>`) and prevented the editor modal and fullscreen control from working correctly. Existing activities configured with *In frame* are migrated automatically to *Embed* on upgrade, and the orphan `framesize` admin setting is removed.
+
 ## v4.0.0 – 2025-04-30
 
 - Version jump to 4.0.0 to align numbering with eXeLearning for consistency across related projects.
diff --git a/db/upgrade.php b/db/upgrade.php
index 922f300..f15b44d 100644
--- a/db/upgrade.php
+++ b/db/upgrade.php
@@ -43,11 +43,21 @@
  */
 
 function xmldb_exeweb_upgrade($oldversion) {
-    global $DB;
+    global $DB, $CFG;
 
     $dbman = $DB->get_manager();
 
-    // Put any upgrade step here.
+    if ($oldversion < 2026050900) {
+        require_once($CFG->libdir . '/resourcelib.php');
+
+        // Frameset display has been removed in favour of the equivalent
+        // IFRAME-based EMBED mode. Migrate legacy activities and drop the
+        // orphan framesize plugin config.
+        $DB->set_field('exeweb', 'display', RESOURCELIB_DISPLAY_EMBED, ['display' => RESOURCELIB_DISPLAY_FRAME]);
+        unset_config('framesize', 'exeweb');
+
+        upgrade_mod_savepoint(true, 2026050900, 'exeweb');
+    }
 
     return true;
 }
diff --git a/lang/ca/exeweb.php b/lang/ca/exeweb.php
index 3479d8c..c4e7c5d 100644
--- a/lang/ca/exeweb.php
+++ b/lang/ca/exeweb.php
@@ -30,7 +30,6 @@
 $string['clicktodownload'] = 'Haga clic en {$a} para descargar el archivo.';
 $string['clicktoopen2'] = 'Haga clic en {$a} para ver el archivo.';
 $string['configdisplayoptions'] = 'Puede seleccionar cualquiera de las opciones disponibles (los ajustes existentes no se modificarán). Mantenga pulsada la tecla CTRL para seleccionar varios campos simultáneamente.';
-$string['configframesize'] = 'Cuando se muestra una página web o un archivo cargado dentro de un marco (FRAME), este valor es el tamaño en píxeles del marco superior, que contiene la navegación.';
 $string['configparametersettings'] = 'Configura el valor por defecto del panel de ajustes cuando se agregan nuevos recursos. Tras esta primera vez, se convierte en una preferencia del usuario.';
 $string['configpopup'] = 'Cuando se agrega un recurso que puede mostrarse en un ventana emergente ("popup"), ¿esta opción se debe activar por defecto?';
 $string['configpopupdirectories'] = 'Las ventanas "popup", ¿deben mostrar por defecto los enlaces del directorio?';
@@ -49,7 +48,6 @@
 $string['displayselect_help'] = 'Este ajuste determina cómo se muestra el contenido. Las opciones pueden incluir:
 
 * Incrustar - Un IFRAME dentro de una página de actividad de Moodle, con sus bloques laterales (dependerá del tema).
-* En el marco - Un FRAMESET con dos marcos. El superior muestra algún dato del curso en el que estás y las opciones para ir a las actividades anterior y siguiente. El inferior muestra el contenido del paquete.
 * Nueva ventana - Se abre en una nueva ventana del navegador.
 * Abrir - Se abre en la misma ventana del navegador.
 * En ventana emergente - Ventana tipo "popup".';
@@ -83,7 +81,6 @@
 $string['filterfilesexplain'] = 'Seleccione el tipo de archivo que contiene el filtro. Esto puede causar problemas en algunos contenidos. Por favor, asegúrese de que todos los archivos de texto están en UTF-8.';
 $string['filtername'] = 'Auto-enlace de nombres de recursos';
 $string['forcedownload'] = 'Forzar descarga';
-$string['framesize'] = 'Altura del marco';
 $string['indicator:cognitivedepth'] = 'Archivo cognitivo';
 $string['indicator:cognitivedepth_help'] = 'Este indicador está basado en la profundidad cognitiva alcanzada por el estudiante.';
 $string['indicator:cognitivedepthdef'] = 'Archivo cognitivo';
diff --git a/lang/en/exeweb.php b/lang/en/exeweb.php
index a963a02..333b72b 100644
--- a/lang/en/exeweb.php
+++ b/lang/en/exeweb.php
@@ -30,7 +30,6 @@
 $string['clicktodownload'] = 'Click {$a} link to download the file.';
 $string['clicktoopen2'] = 'Click {$a} link to view the file.';
 $string['configdisplayoptions'] = 'Select all options that should be available, existing settings are not modified. Hold CTRL key to select multiple fields.';
-$string['configframesize'] = 'When a web page or an uploaded file is displayed within a frame, this value is the height (in pixels) of the top frame (which contains the navigation).';
 $string['configparametersettings'] = 'This sets the default value for the Parameter settings pane in the form when adding some new contents. After the first time, this becomes an individual user preference.';
 $string['configpopup'] = 'When adding a new content that can be shown in a popup window, should this option be enabled by default?';
 $string['configpopupdirectories'] = 'Should popup windows show directory links by default?';
@@ -49,7 +48,6 @@
 $string['displayselect_help'] = 'This setting determines how the content is displayed. Options may include:
 
 * Embed - The file is displayed in an IFRAME, below the navigation bar, together with the file description and any blocks.
-* In frame - The file is displayed using a FRAMESET with two FRAMES, below the navigation bar and the file description.
 * New window - The content is open on a new browser window.
 * Open - The content is open on the same browser window.
 * In pop-up - The file is displayed in a new browser window without menus or address bar (popup window).';
@@ -83,7 +81,6 @@
 $string['filterfilesexplain'] = 'Select type of file content filtering, please note this may cause problems in some contents. Please make sure that all text files are in UTF-8 encoding.';
 $string['filtername'] = 'Content names auto-linking';
 $string['forcedownload'] = 'Force download';
-$string['framesize'] = 'Frame height';
 $string['indicator:cognitivedepth'] = 'File cognitive';
 $string['indicator:cognitivedepth_help'] = 'This indicator is based on the cognitive depth reached by the student.';
 $string['indicator:cognitivedepthdef'] = 'File cognitive';
diff --git a/lang/es/exeweb.php b/lang/es/exeweb.php
index 9673290..d0a44ac 100644
--- a/lang/es/exeweb.php
+++ b/lang/es/exeweb.php
@@ -30,7 +30,6 @@
 $string['clicktodownload'] = 'Haga clic en {$a} para descargar el archivo.';
 $string['clicktoopen2'] = 'Haga clic en {$a} para ver el archivo.';
 $string['configdisplayoptions'] = 'Puede seleccionar cualquiera de las opciones disponibles (los ajustes existentes no se modificarán). Mantenga pulsada la tecla CTRL para seleccionar varios campos simultáneamente.';
-$string['configframesize'] = 'Cuando se muestra una página web o un archivo cargado dentro de un marco (FRAME), este valor es el tamaño en píxeles del marco superior, que contiene la navegación.';
 $string['configparametersettings'] = 'Configura el valor por defecto del panel de ajustes cuando se agregan nuevos recursos. Tras esta primera vez, se convierte en una preferencia del usuario.';
 $string['configpopup'] = 'Cuando se agrega un recurso que puede mostrarse en un ventana emergente ("popup"), ¿esta opción se debe activar por defecto?';
 $string['configpopupdirectories'] = 'Las ventanas "popup", ¿deben mostrar por defecto los enlaces del directorio?';
@@ -49,7 +48,6 @@
 $string['displayselect_help'] = 'Este ajuste determina cómo se muestra el contenido. Las opciones pueden incluir:
 
 * Incrustar - Un IFRAME dentro de una página de actividad de Moodle, con sus bloques laterales (dependerá del tema).
-* En el marco - Un FRAMESET con dos marcos. El superior muestra algún dato del curso en el que estás y las opciones para ir a las actividades anterior y siguiente. El inferior muestra el contenido del paquete.
 * Nueva ventana - Se abre en una nueva ventana del navegador.
 * Abrir - Se abre en la misma ventana del navegador.
 * En ventana emergente - Ventana tipo "popup".';
@@ -83,7 +81,6 @@
 $string['filterfilesexplain'] = 'Seleccione el tipo de archivo que contiene el filtro. Esto puede causar problemas en algunos contenidos. Por favor, asegúrese de que todos los archivos de texto están en UTF-8.';
 $string['filtername'] = 'Auto-enlace de nombres de recursos';
 $string['forcedownload'] = 'Forzar descarga';
-$string['framesize'] = 'Altura del marco';
 $string['indicator:cognitivedepth'] = 'Archivo cognitivo';
 $string['indicator:cognitivedepth_help'] = 'Este indicador está basado en la profundidad cognitiva alcanzada por el estudiante.';
 $string['indicator:cognitivedepthdef'] = 'Archivo cognitivo';
diff --git a/lang/eu/exeweb.php b/lang/eu/exeweb.php
index 425d035..7cdc298 100644
--- a/lang/eu/exeweb.php
+++ b/lang/eu/exeweb.php
@@ -30,7 +30,6 @@
 $string['clicktodownload'] = 'Haga clic en {$a} para descargar el archivo.';
 $string['clicktoopen2'] = 'Haga clic en {$a} para ver el archivo.';
 $string['configdisplayoptions'] = 'Puede seleccionar cualquiera de las opciones disponibles (los ajustes existentes no se modificarán). Mantenga pulsada la tecla CTRL para seleccionar varios campos simultáneamente.';
-$string['configframesize'] = 'Cuando se muestra una página web o un archivo cargado dentro de un marco (FRAME), este valor es el tamaño en píxeles del marco superior, que contiene la navegación.';
 $string['configparametersettings'] = 'Configura el valor por defecto del panel de ajustes cuando se agregan nuevos recursos. Tras esta primera vez, se convierte en una preferencia del usuario.';
 $string['configpopup'] = 'Cuando se agrega un recurso que puede mostrarse en un ventana emergente ("popup"), ¿esta opción se debe activar por defecto?';
 $string['configpopupdirectories'] = 'Las ventanas "popup", ¿deben mostrar por defecto los enlaces del directorio?';
@@ -49,7 +48,6 @@
 $string['displayselect_help'] = 'Este ajuste determina cómo se muestra el contenido. Las opciones pueden incluir:
 
 * Incrustar - Un IFRAME dentro de una página de actividad de Moodle, con sus bloques laterales (dependerá del tema).
-* En el marco - Un FRAMESET con dos marcos. El superior muestra algún dato del curso en el que estás y las opciones para ir a las actividades anterior y siguiente. El inferior muestra el contenido del paquete.
 * Nueva ventana - Se abre en una nueva ventana del navegador.
 * Abrir - Se abre en la misma ventana del navegador.
 * En ventana emergente - Ventana tipo "popup".';
@@ -83,7 +81,6 @@
 $string['filterfilesexplain'] = 'Seleccione el tipo de archivo que contiene el filtro. Esto puede causar problemas en algunos contenidos. Por favor, asegúrese de que todos los archivos de texto están en UTF-8.';
 $string['filtername'] = 'Auto-enlace de nombres de recursos';
 $string['forcedownload'] = 'Forzar descarga';
-$string['framesize'] = 'Altura del marco';
 $string['indicator:cognitivedepth'] = 'Archivo cognitivo';
 $string['indicator:cognitivedepth_help'] = 'Este indicador está basado en la profundidad cognitiva alcanzada por el estudiante.';
 $string['indicator:cognitivedepthdef'] = 'Archivo cognitivo';
diff --git a/lang/gl/exeweb.php b/lang/gl/exeweb.php
index fdcdd65..0592538 100644
--- a/lang/gl/exeweb.php
+++ b/lang/gl/exeweb.php
@@ -30,7 +30,6 @@
 $string['clicktodownload'] = 'Faga clic en {$a} para descargar o ficheiro.';
 $string['clicktoopen2'] = 'Faga clic en {$a} para ver o ficheiro.';
 $string['configdisplayoptions'] = 'Pode seleccionar calquera das opcións dispoñibles (os axustes existentes non se modificarán). Manteña premida a tecla CTRL para seleccionar varios campos simultaneamente.';
-$string['configframesize'] = 'Cando se amosa unha páxina web ou un ficheiro cargado dentro dun marco (FRAME), este valor é o tamaño en píxeles do marco superior que contén a navegación.';
 $string['configparametersettings'] = 'Configura o valor por defecto do panel de axustes cando se agregan novos recursos. Logo desta primeira vez, convértese nunha preferencia do usuario.';
 $string['configpopup'] = 'Cando se agrega un recurso que pode amosarse nunha xanela emerxente ("popup"), esta opción débese activar por defecto?';
 $string['configpopupdirectories'] = 'As xanelas "popup", deben amosar por defecto as ligazóns do directorio?';
@@ -49,7 +48,6 @@
 $string['displayselect_help'] = 'Este axuste determina como se amosa o contido. As opcións poden incluír:
 
 * Incrustar - Un IFRAME dentro dunha páxina de actividade de Moodle, cos seus bloques laterais (dependerá do estilo de eXeLearning).
-* No marco - Un FRAMESET con dous marcos. O superior amosa algún dato do curso no que estás e as opcións para ir ás actividades anterior e seguinte. O inferior amosa o contido do paquete.
 * Nova xanela - Ábrese unha nova xanela do navegador.
 * Abrir - Ábrese na mesma xanela do navegador.
 * En xanela emerxente - Xanela tipo "popup".';
@@ -83,7 +81,6 @@
 $string['filterfilesexplain'] = 'Seleccione o tipo de ficheiro que contén o filtro. Esto pode causar problemas nalgúns contidos. Por favor, revise que todos os ficheiros de texto están en UTF-8.';
 $string['filtername'] = 'Auto-ligazón de nomes de recursos';
 $string['forcedownload'] = 'Forzar descarga';
-$string['framesize'] = 'Altura do marco';
 $string['indicator:cognitivedepth'] = 'Ficheiro cognitivo';
 $string['indicator:cognitivedepth_help'] = 'Este indicador está baseado na profundidade cognitiva acadada polo estudante.';
 $string['indicator:cognitivedepthdef'] = 'Ficheiro cognitivo';
diff --git a/lib.php b/lib.php
index 58f1ec9..0045992 100644
--- a/lib.php
+++ b/lib.php
@@ -229,7 +229,7 @@ function exeweb_set_display_options($data) {
         $displayoptions['popupwidth']  = $data->popupwidth;
         $displayoptions['popupheight'] = $data->popupheight;
     }
-    if (in_array($data->display, [RESOURCELIB_DISPLAY_EMBED, RESOURCELIB_DISPLAY_FRAME])) {
+    if ($data->display == RESOURCELIB_DISPLAY_EMBED) {
         $displayoptions['printintro']   = (int)!empty($data->printintro);
     }
     if (!empty($data->showsize)) {
diff --git a/locallib.php b/locallib.php
index c0d44c6..a1c9899 100644
--- a/locallib.php
+++ b/locallib.php
@@ -67,68 +67,6 @@ function exeweb_display_embed($exeweb, $cm, $course, $file) {
     die;
 }
 
-
-
-
-/**
- * Display exeweb frames.
- * @param object $exeweb
- * @param object $cm
- * @param object $course
- * @param stored_file $file main file
- * @return does not return
- */
-function exeweb_display_frame($exeweb, $cm, $course, $file) {
-    global $PAGE, $OUTPUT, $CFG;
-
-    $frame = optional_param('frameset', 'main', PARAM_ALPHA);
-
-    if ($frame === 'top') {
-        $PAGE->set_pagelayout('frametop');
-        $PAGE->activityheader->set_description(exeweb_get_intro($exeweb, $cm, true));
-        exeweb_print_header($exeweb, $cm, $course);
-        if (!exeweb_is_teacher_mode_visible($exeweb)) {
-            exeweb_require_teacher_mode_hider_for_content_frame();
-        }
-        echo $OUTPUT->footer();
-        die;
-
-    } else {
-        $config = get_config('exeweb');
-        $context = context_module::instance($cm->id);
-        $fileurl = moodle_url::make_pluginfile_url($context->id, 'mod_exeweb', 'content', $exeweb->revision,
-                        $file->get_filepath(), $file->get_filename());
-        $navurl = "$CFG->wwwroot/mod/exeweb/view.php?id=$cm->id&amp;frameset=top";
-        $title = strip_tags(format_string($course->shortname.': '.$exeweb->name));
-        $framesize = $config->framesize;
-        $contentframetitle = s(format_string($exeweb->name));
-        $modulename = s(get_string('modulename', 'mod_exeweb'));
-        $dir = get_string('thisdirection', 'langconfig');
-
-        $file = <<<EOF
-<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Frameset//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-frameset.dtd">
-<html dir="$dir">
-  <head>
-    <meta http-equiv="content-type" content="text/html; charset=utf-8" />
-    <title>$title</title>
-  </head>
-  <frameset rows="$framesize,*">
-    <frame src="$navurl" title="$modulename" />
-    <frame src="$fileurl" title="$contentframetitle" />
-  </frameset>
-</html>
-EOF;
-
-        @header('Content-Type: text/html; charset=utf-8');
-        echo $file;
-        die;
-    }
-}
-
-/**
- * Internal function - create click to open text with link.
- */
-
 /**
  * Check whether teacher mode toggler should be visible for this activity.
  *
@@ -168,27 +106,6 @@ function exeweb_require_teacher_mode_hider_for_iframe(string $iframeid): void {
     $PAGE->requires->js_init_code($js);
 }
 
-/**
- * Inject CSS into the frame content document to hide teacher mode toggler.
- *
- * @return void
- */
-function exeweb_require_teacher_mode_hider_for_content_frame(): void {
-    global $PAGE;
-
-    $cssjson = json_encode('#teacher-mode-toggler-wrapper { visibility: hidden !important; }');
-    $js = "(function(){"
-        . "var css=" . $cssjson . ";"
-        . "var inject=function(){try{if(!window.parent||!window.parent.frames||!window.parent.frames[1]){return;}"
-        . "var frameWin=window.parent.frames[1];if(!frameWin.document){return;}"
-        . "var d=frameWin.document;var st=d.createElement('style');st.textContent=css;"
-        . "(d.head||d.documentElement).appendChild(st);}catch(e){}};"
-        . "window.addEventListener('load', inject);setTimeout(inject, 300);"
-        . "})();";
-
-    $PAGE->requires->js_init_code($js);
-}
-
 function exeweb_get_clicktoopen($file, $revision, $extra='') {
     global $CFG;
 
@@ -218,6 +135,9 @@ function exeweb_print_workaround($exeweb, $cm, $course, $file) {
 
     exeweb_print_header($exeweb, $cm, $course);
 
+    // Show action bar with Edit button when user has edit capability.
+    echo $PAGE->get_renderer('mod_exeweb')->generate_action_bar($cm);
+
     echo '<div class="exewebworkaround">';
     switch ($exeweb->display) {
         case RESOURCELIB_DISPLAY_POPUP:
diff --git a/mod_form.php b/mod_form.php
index c82b280..b62d5a3 100644
--- a/mod_form.php
+++ b/mod_form.php
@@ -158,8 +158,7 @@ public function definition() {
             $mform->setAdvanced('popupheight', true);
         }
 
-        if (array_key_exists(RESOURCELIB_DISPLAY_EMBED, $options) ||
-          array_key_exists(RESOURCELIB_DISPLAY_FRAME, $options)) {
+        if (array_key_exists(RESOURCELIB_DISPLAY_EMBED, $options)) {
             $mform->addElement('checkbox', 'printintro', get_string('printintro', 'mod_exeweb'));
             $mform->hideIf('printintro', 'display', 'eq', RESOURCELIB_DISPLAY_POPUP);
             $mform->hideIf('printintro', 'display', 'eq', RESOURCELIB_DISPLAY_OPEN);
diff --git a/settings.php b/settings.php
index 32774b8..ec4fa9b 100644
--- a/settings.php
+++ b/settings.php
@@ -185,7 +185,6 @@ function toggleConnectionSettings() {
 
     $displayoptions = resourcelib_get_displayoptions([
         RESOURCELIB_DISPLAY_EMBED,
-        RESOURCELIB_DISPLAY_FRAME,
         RESOURCELIB_DISPLAY_OPEN,
         RESOURCELIB_DISPLAY_NEW,
         RESOURCELIB_DISPLAY_POPUP,
@@ -196,9 +195,6 @@ function toggleConnectionSettings() {
         RESOURCELIB_DISPLAY_POPUP,
     ];
 
-    // General settings.
-    $settings->add(new admin_setting_configtext('exeweb/framesize',
-        new lang_string('framesize', 'mod_exeweb'), new lang_string('configframesize', 'mod_exeweb'), 130, PARAM_INT));
     $settings->add(new admin_setting_configmultiselect('exeweb/displayoptions',
         new lang_string('displayoptions', 'mod_exeweb'), new lang_string('configdisplayoptions', 'mod_exeweb'),
         $defaultdisplayoptions, $displayoptions));
diff --git a/view.php b/view.php
index 7c8f1a5..6e0394e 100644
--- a/view.php
+++ b/view.php
@@ -91,10 +91,9 @@
 $PAGE->requires->js_call_amd('mod_exeweb/editor_modal', 'init');
 switch ($displaytype) {
     case RESOURCELIB_DISPLAY_EMBED:
-        exeweb_display_embed($exeweb, $cm, $course, $file);
-        break;
     case RESOURCELIB_DISPLAY_FRAME:
-        exeweb_display_frame($exeweb, $cm, $course, $file);
+        // FRAME mode is deprecated; legacy activities fall back to EMBED.
+        exeweb_display_embed($exeweb, $cm, $course, $file);
         break;
     default:
         exeweb_print_workaround($exeweb, $cm, $course, $file);

From 284d34f4043dc11fd2637050f3e0375a8d3c1b04 Mon Sep 17 00:00:00 2001
From: erseco <info@ernesto.es>
Date: Wed, 3 Jun 2026 18:52:20 +0100
Subject: [PATCH 2/2] fix: normalise legacy FRAME display to EMBED in the
 activity form

The DB upgrade in db/upgrade.php only converts display=FRAME rows when the
plugin version actually increases, which never happens on git/dev installs
(version.php ships the 9999999999 placeholder overridden by CI). On those
installs an activity created with the removed *In frame* mode still showed
the In frame option (value=2) in the edit form's display dropdown.

Normalise FRAME -> EMBED in mod_form (both when building the display options
and in data_preprocessing) so the removed option never reappears and the
field defaults to Embed, independently of whether the DB migration ran.
The upgrade migration is kept for real released-version upgrades.
---
 CHANGELOG.md |  2 +-
 mod_form.php | 13 ++++++++++++-
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 8ab50ef..bb09cf2 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -3,7 +3,7 @@
 ## Unreleased
 
 - Show the eXeLearning **Edit** button on the activity view when **Display** is set to *In pop-up*, *Open* or *New window* (issue #43). Previously the button only appeared for *Embed* activities, leaving teachers no way to edit the package from those modes.
-- Remove the legacy *In frame* (HTML4 frameset) display option. The mode duplicated *Embed* (which already uses an `<iframe>`) and prevented the editor modal and fullscreen control from working correctly. Existing activities configured with *In frame* are migrated automatically to *Embed* on upgrade, and the orphan `framesize` admin setting is removed.
+- Remove the legacy *In frame* (HTML4 frameset) display option. The mode duplicated *Embed* (which already uses an `<iframe>`) and prevented the editor modal and fullscreen control from working correctly. Existing activities configured with *In frame* are migrated to *Embed* on upgrade; the activity form also normalises any leftover *In frame* value to *Embed* when edited, so the removed option never reappears in the dropdown. The orphan `framesize` admin setting is removed.
 
 ## v4.0.0 – 2025-04-30
 
diff --git a/mod_form.php b/mod_form.php
index b62d5a3..eaac867 100644
--- a/mod_form.php
+++ b/mod_form.php
@@ -120,7 +120,13 @@ public function definition() {
         $mform->addElement('header', 'optionssection', get_string('appearance'));
 
         if ($this->current->instance) {
-            $options = resourcelib_get_displayoptions(explode(',', $config->displayoptions), $this->current->display);
+            // Legacy FRAME activities are rendered as EMBED; normalise the stored value so the
+            // removed "In frame" option is not re-injected into the dropdown for those activities.
+            $currentdisplay = (int) $this->current->display;
+            if ($currentdisplay === RESOURCELIB_DISPLAY_FRAME) {
+                $currentdisplay = RESOURCELIB_DISPLAY_EMBED;
+            }
+            $options = resourcelib_get_displayoptions(explode(',', $config->displayoptions), $currentdisplay);
         } else {
             $options = resourcelib_get_displayoptions(explode(',', $config->displayoptions));
         }
@@ -191,6 +197,11 @@ public function definition() {
 
 
     public function data_preprocessing(&$defaultvalues) {
+        // Migrate legacy FRAME activities to EMBED so the select defaults correctly and never
+        // shows the removed "In frame" option (the DB upgrade only covers real version bumps).
+        if (isset($defaultvalues['display']) && (int) $defaultvalues['display'] === RESOURCELIB_DISPLAY_FRAME) {
+            $defaultvalues['display'] = RESOURCELIB_DISPLAY_EMBED;
+        }
         if ($this->current->instance) {
             $draftitemid = file_get_submitted_draft_itemid('packagefile');
             file_prepare_draft_area($draftitemid, $this->context->id, 'mod_exeweb', 'package',