Move RML generation to Editor:Finalize() method

Fixed architecture - RML is now generated when editor initializes, not when opened:

editor.lua changes:
- Editor:Finalize() now detects RmlUi mode and calls _FinalizeRmlUi()
- _FinalizeRmlUi() loops through fields, calls GenerateRml(), stores result
- Generated RML stored in editor.generatedRml
- Added ShowRmlUi() and HideRmlUi() methods for visibility

view.lua changes:
- OpenEditor() now uses pre-generated editor.generatedRml
- No longer generates RML on-the-fly
- Much cleaner - just displays what was already generated

This is the correct flow:
1. Editor constructor calls init()
2. Editor adds fields with AddField()
3. Editor calls Finalize() which:
   - RmlUi mode: generates RML → editor.generatedRml
   - Chili mode: creates Chili controls → editor.window
4. User clicks button → OpenEditor() displays editor.generatedRml

All field generation happens during editor initialization!

Author: claude

scen_edit/view/editor.lua

@@ -115,6 +115,15 @@ function Editor:Finalize(children, opts)
     end
 
     opts = opts or {}
+
+    -- In RmlUi mode, generate RML from fields instead of creating Chili controls
+    if SB.view and SB.view.useRmlUi then
+        self:_FinalizeRmlUi(opts)
+        self.__initializing = false
+        return
+    end
+
+    -- Chili mode continues below
     self:_FinalizeButtons(children, opts)
 
     local OnShow = {function() self:__OnShow() end}
@@ -655,3 +664,44 @@ end
 
 -- We load these fields last as they might be/contain subclasses of editor view
 SB.IncludeDir(Path.Join(SB.DIRS.SRC, 'view/fields'))
+
+-- RmlUi-specific finalization
+function Editor:_FinalizeRmlUi(opts)
+    -- Generate RML for all fields
+    local html = ''
+    
+    for _, fieldName in ipairs(self.fieldOrder) do
+        local field = self.fields[fieldName]
+        if field and field.GenerateRml then
+            html = html .. field:GenerateRml()
+        elseif field then
+            -- Fallback for fields without GenerateRml (like separators/controls)
+            html = html .. '<div class="field-separator"></div>'
+        end
+    end
+    
+    -- Store the generated RML for later use
+    self.generatedRml = html
+    
+    -- Mark as hidden by default
+    self.hidden = true
+end
+
+-- Show the editor in RmlUi mode
+function Editor:ShowRmlUi()
+    self.hidden = false
+    if SB.view and SB.view.OpenEditor then
+        -- Trigger view to display this editor
+        for name, editor in pairs(SB.editors) do
+            if editor == self then
+                SB.view:DisplayEditor(name)
+                break
+            end
+        end
+    end
+end
+
+-- Hide the editor in RmlUi mode
+function Editor:HideRmlUi()
+    self.hidden = true
+end

scen_edit/view/view.lua

@@ -440,26 +440,26 @@ function View:OpenEditor(editorName)
     local editor = SB.editors[editorName]
     local editorCfg = SB.editorRegistry[editorName]
     local mainContent = self.mainDocument:GetElementById("main-content")
+
     if mainContent and editor then
-        -- Render all fields to RML
-        local html = '<div class="editor-container"><h3 class="editor-title">' .. (editorCfg and editorCfg.caption or editorName) .. '</h3>'
-
-        -- Generate RML for each field in order
-        for _, fieldName in ipairs(editor.fieldOrder) do
-            local field = editor.fields[fieldName]
-            if field and field.GenerateRml then
-                html = html .. field:GenerateRml()
-            elseif field then
-                -- Fallback for fields without GenerateRml (like separators)
-                html = html .. '<div class="field-separator"></div>'
-            end
+        -- Use pre-generated RML from Finalize()
+        local html = '<div class="editor-container"><h3 class="editor-title">' ..
+                     (editorCfg and editorCfg.caption or editorName) .. '</h3>'
+
+        if editor.generatedRml then
+            html = html .. editor.generatedRml
+        else
+            html = html .. '<p>Editor has no fields defined</p>'
         end
 
         html = html .. '</div>'
         mainContent.inner_rml = html
 
         -- Bind field events
         self:BindFieldEvents(editor)
+
+        -- Mark editor as visible
+        editor.hidden = false
     end
 
     -- Highlight pressed button