Fix RmlUi field generation - add fields to fieldOrder

CRITICAL FIX: RmlUi fields weren't being added to fieldOrder array

The problem:
- _AddControl() adds to fieldOrder (for Chili fields with components)
- _AddField() did NOT add to fieldOrder
- RmlUi fields don't have .components, so they only call _AddField()
- Result: fieldOrder was empty, _FinalizeRmlUi() generated no HTML

The fix:
- Modified _AddField() to add field.name to fieldOrder
- Checks for duplicates (in case field was already added via _AddControl)
- Now RmlUi fields appear in fieldOrder and get rendered

Also:
- Removed no_serialize skip in InitializeAllEditors()
- no_serialize means "don't save state", not "don't initialize"
- objectPropertyWindow and collisionView are valid editors that need initialization

This fixes: "all editors have 0 fields" - they now generate proper RML

Author: claude

scen_edit/view/editor.lua

@@ -387,6 +387,19 @@ function Editor:_AddField(field)
     end
     self.fields[field.name] = field
     field.ev = self
+
+    -- Add to fieldOrder if not already added (via _AddControl)
+    -- This is needed for RmlUi fields which don't have components
+    local alreadyInOrder = false
+    for _, name in ipairs(self.fieldOrder) do
+        if name == field.name then
+            alreadyInOrder = true
+            break
+        end
+    end
+    if not alreadyInOrder then
+        table.insert(self.fieldOrder, field.name)
+    end
 end
 
 function Editor:AddControl(name, children)

scen_edit/view/view.lua

@@ -172,16 +172,13 @@ function View:InitializeAllEditors()
     -- Instantiate all editors from editorRegistry
     -- This calls their init() which calls Finalize() which generates RML
     -- Doing this upfront catches field errors early instead of when users click buttons
-    -- Skip editors with no_serialize=true (utility windows created on-demand)
     Log.Notice("Initializing all editors from editorRegistry...")
     local count = 0
     for name, editorCfg in pairs(SB.editorRegistry) do
-        if editorCfg.editor and not editorCfg.no_serialize then
+        if editorCfg.editor then
             Log.Notice("  Creating editor: " .. name)
             SB.editors[name] = editorCfg.editor()
             count = count + 1
-        elseif editorCfg.no_serialize then
-            Log.Notice("  Skipping utility window: " .. name .. " (no_serialize)")
         else
             Log.Warning("  Editor " .. name .. " has no constructor function")
         end