You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
- Banner is after the shared header partial and before `main`.
47
+
- Banner is a single clickable status row.
48
+
- Guest, creator, and admin role modes remain supported.
49
+
- Banner uses existing Theme V2 `callout` and `status` classes.
50
+
51
+
## Skipped Changes
52
+
53
+
- No Project Workspace implementation was started.
54
+
- No DB, auth, persistence, or save/load behavior was added.
55
+
- No new tools were added.
56
+
- No CSS was added or modified.
57
+
58
+
## Validation Notes
59
+
60
+
- First validation run exposed two issues:
61
+
- The partial loader keeps the header inside its placeholder wrapper, so the role-banner position assertion was adjusted to match the actual Theme V2 partial structure.
62
+
- Old standalone brand-suffix copy existed on common pages; those text-only labels were updated.
63
+
- Final validation:
64
+
-`npm run test:workspace-v2` passed with 4 Playwright tests.
65
+
- Common header pages had no page errors or console errors.
Copy file name to clipboardExpand all lines: docs_build/dev/reports/navigation-and-list-sorting-rule.md
+1-1Lines changed: 1 addition & 1 deletion
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -5,7 +5,6 @@
5
5
Added governance requiring navigation menus, submenus, nested submenus, and browseable user-facing lists to sort alphabetically.
6
6
7
7
Normalized safe active surfaces:
8
-
- Header top-level navigation.
9
8
- Header Games submenu.
10
9
- Header Toolbox group order.
11
10
- Header Toolbox nested tool submenus.
@@ -17,6 +16,7 @@ Normalized safe active surfaces:
17
16
## Sorting Exceptions
18
17
19
18
Documented intentional-order exceptions:
19
+
- Header top-level navigation is intentionally ordered by product IA: Games, Toolbox, Marketplace, Learn, Account, Admin.
20
20
- Build Path remains dependency ordered: Project Workspace, Game Design, Game Configuration, Required Tool Path, Build Game, Game Testing, Publish.
21
21
- Project Progress and Publishing Progress remain progress/workflow ordered.
22
22
- Individual Learn tool page section order remains a guided documentation sequence: Overview, Quick Start, Common Tasks, Related Documentation, Related Videos, Examples.
Set the primary top-level header navigation order exactly:
6
+
- Games
7
+
- Toolbox
8
+
- Marketplace
9
+
- Learn
10
+
- Account
11
+
- Admin
12
+
13
+
This order is now documented in `docs_build/dev/PROJECT_INSTRUCTIONS.md` as an explicit product information architecture exception to the alphabetical navigation rule.
14
+
15
+
## Sorting Rule Impact
16
+
17
+
Alphabetical sorting still applies to:
18
+
- Header submenus.
19
+
- Toolbox nested submenus.
20
+
- Account menu links.
21
+
- Admin menu links.
22
+
- Games menu links.
23
+
- Footer link lists.
24
+
- Browseable Learn lists.
25
+
- Browseable Toolbox group mode.
26
+
27
+
Documented intentional-order exceptions:
28
+
- Primary top-level header navigation.
29
+
- Workflow paths.
30
+
- Build Path.
31
+
- Dependency paths.
32
+
- Project Progress.
33
+
- Publishing Progress.
34
+
- Guided creator steps.
35
+
36
+
## Additional Text Cleanup
37
+
38
+
The common header validation found old standalone brand-suffix copy on the exact active pages under test. These labels were updated to `GameFoundryStudio` or current creator-facing names:
39
+
-`index.html`
40
+
-`games/index.html`
41
+
-`marketplace/index.html`
42
+
-`account/index.html`
43
+
-`admin/site-settings.html`
44
+
45
+
## Manual Test Notes
46
+
47
+
- Header order is Games, Toolbox, Marketplace, Learn, Account, Admin.
48
+
- Toolbox submenu groups remain alphabetical.
49
+
- Toolbox nested menu items remain alphabetical.
50
+
- Account, Admin, and Games submenu links remain alphabetical.
51
+
- Admin is not duplicated under Toolbox.
52
+
- Arcade remains a Games menu item only and is absent from Toolbox.
53
+
54
+
## Validation Notes
55
+
56
+
-`npm run test:workspace-v2` passed with 4 Playwright tests.
0 commit comments