Skip to content

Commit da8ced6

Browse files
committed
testing notes for layout config - windows not validated
1 parent a2fa6b4 commit da8ced6

1 file changed

Lines changed: 12 additions & 0 deletions

File tree

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
# New behavior
2+
3+
Startup: App reads default_layout.json from the platform settings directory. If valid, sets mLayoutPath to that path and logs success. If missing, continues silently. If present but invalid/unreadable, logs a non-fatal amber warning and continues without a layout.
4+
"Set as Default" button: reads current mLayoutPath, validates the JSON, atomically copies to settings dir, writes metadata. Appears disabled when no layout is selected.
5+
"Clear Default" button: removes default_layout.json and .meta.json; never touches the original user file. Appears disabled when no default is saved.
6+
Status indicator: shows one of none saved / loaded (name, saved-at) / saved file invalid / unavailable.
7+
8+
# Follow-up recommendations
9+
10+
Tests: The SPATIALROOT_SETTINGS_ROOT env override makes DefaultLayoutManager fully testable in a temp dir without touching the real settings dir — a CTest unit covering save/load/clear/invalid-JSON would be straightforward to add.
11+
Windows testing: The %APPDATA% path branch hasn't been exercised on a real Windows build yet.
12+
Schema version in metadata: the task spec mentions schemaVersion and appVersion fields — these can be added to the metadata write in saveDefaultLayout() once those values are surfaced from the engine or build system.

0 commit comments

Comments
 (0)