Bug: al cargar un layout guardado, paneles que estaban dockeados a la
ventana principal aparecian flotantes. Causa: drain_layout_pending()
(que llama ImGui::LoadIniSettingsFromMemory) corria mid-frame dentro
de render(), DESPUES de menubar y de auto-dockspace. ImGui requiere
que el load suceda ANTES de cualquier Begin() del frame para que las
dock-nodes guardadas se restauren correctamente.
Fix:
- Mover drain_layout_pending al hook cfg.pre_frame del framework, que
se ejecuta entre NewFrame y menubar (orden correcto). Requiere el
commit asociado en cpp/framework/app_base.{h,cpp} que añade el hook.
- Eliminar la llamada redundante a ImGui::DockSpaceOverViewport en
render(): el framework ya la hace via cfg.auto_dockspace=true.
dashboard_state.h: declara render() para que tests puedan referenciarla
con `extern "C"`-like visibility (consistente con resto del API).
tests/: 7/7 siguen pasando.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Bug: al pulsar un layout guardado el dock layout se aplicaba pero los paneles
ocultos quedaban con dock node vacio. Causa: el INI de ImGui solo guarda
posicion/dock; la visibilidad (`show_*` bools) es estado puro de la app y no
se restauraba al apply.
Fix:
- Tomar control del menu Layouts (cfg.auto_layouts=false). Abrir LayoutStorage
propio + segunda tabla `panel_visibility` en la misma layouts.db (CREATE
TABLE IF NOT EXISTS, aditivo, no rompe layouts existentes).
- on_save: capture_panel_state() serializa show_* a JSON y se persiste junto
al INI bajo el mismo nombre.
- on_apply: marca pending INI + carga state JSON pendiente.
- on_reset: clear INI + open_all_panels (reabre Browsers/Tabs/TabDetail/Network).
- on_delete: borra fila imgui_layouts + sidecar.
- drain_layout_pending() (llamado desde render() cada frame) aplica
LoadIniSettingsFromMemory + apply_panel_state. Fallback: si layout no tiene
sidecar (back-compat con layouts antiguos) abre todos los paneles.
Refactor:
- main.cpp: render() ya no es static — necesario para que el test harness
reuse la misma funcion. int main() guardado tras `#ifndef FN_TEST_BUILD`.
- show_* bools y k_panels movidos al namespace navegator (extern para tests).
- dashboard_state.h: nuevo header expone show_*, setup_layouts(),
teardown_layouts(), capture/apply_panel_state, open_all_panels y los
hooks layout_save/apply/delete/reset + drain_layout_pending para tests.
Tests (Dear ImGui Test Engine, opt-in via -DFN_BUILD_TESTS=ON):
tests/navegator_dashboard_tests.cpp — 6 tests, todos pasan:
1. panel_state_roundtrip — capture/apply JSON simetrico.
2. open_all_panels_marks_main_visible.
3. save_hide_apply_restores_visibility (FIX BUG).
4. two_layouts_swap_visibility — minimal vs full.
5. reset_opens_all_main_panels.
6. legacy_layout_fallback_opens_all — sin sidecar.
Build/run:
cmake -B cpp/build/windows_tests -S cpp \
-DCMAKE_TOOLCHAIN_FILE=$(pwd)/cpp/toolchains/mingw-w64.cmake \
-DFN_BUILD_TESTS=ON
cmake --build cpp/build/windows_tests --target navegator_dashboard_tests
Deploy + run via cmd.exe -> 6/6 tests passed.
CMakeLists.txt: añade target navegator_dashboard_tests bajo if(FN_BUILD_TESTS),
linka mismas libs que prod + define FN_TEST_BUILD para que main.cpp no
duplique main(). WIN32_EXECUTABLE FALSE para ver stdout en consola.
Issue 0003 (sub-issue del roadmap navegator_dashboard 0001).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
egutierrez
fn-registry agent