egutierrez
df8c0864f6
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>
46 lines
1.7 KiB
C++
46 lines
1.7 KiB
C++
#pragma once
|
|
|
|
// Estado interno expuesto para tests e2e (Dear ImGui Test Engine) y para que
|
|
// main.cpp y el test harness compartan el wiring de layouts.
|
|
|
|
#include "app_base.h"
|
|
|
|
#include <string>
|
|
|
|
namespace navegator {
|
|
|
|
// Bools de visibilidad de cada panel (sincronizados con k_panels en main.cpp).
|
|
extern bool show_browsers;
|
|
extern bool show_tabs;
|
|
extern bool show_tab_detail;
|
|
extern bool show_network;
|
|
extern bool show_agent;
|
|
|
|
// Cablea el menu Layouts: abre `<local_dir>/layouts.db`, monta callbacks que
|
|
// persisten ademas la visibilidad de paneles, y setea `cfg.layouts_cb` +
|
|
// `cfg.auto_layouts = false`. Idempotente — no-op si ya estaba inicializado.
|
|
void setup_layouts(fn::AppConfig& cfg);
|
|
|
|
// Cierra los handles de SQLite. Llamar tras `run_app` / `run_app_test`.
|
|
void teardown_layouts();
|
|
|
|
// Helpers usados tambien por los tests para verificar el comportamiento del
|
|
// fix: capturar/aplicar el JSON de visibilidad y abrir todos los paneles.
|
|
std::string capture_panel_state();
|
|
void apply_panel_state(const std::string& json);
|
|
void open_all_panels();
|
|
|
|
// Hooks que los tests usan para invocar el flujo de layouts SIN tocar la UI
|
|
// (MainMenuBar/popups son flaky bajo Test Engine en algunos drivers OpenGL).
|
|
// Equivalentes a los callbacks que cablearia el menu Layouts.
|
|
bool layout_save(const std::string& name); // captura INI + estado
|
|
bool layout_apply(const std::string& name); // marca pending
|
|
void layout_reset(); // clear INI + abrir todo
|
|
bool layout_delete(const std::string& name); // borra fila + sidecar
|
|
|
|
// Drena el pending del LayoutStorage del navegator_dashboard. Llamar desde
|
|
// render(); aqui expuesto para que tests puedan forzarlo entre frames.
|
|
std::string drain_layout_pending();
|
|
|
|
} // namespace navegator
|