Skip to content

Commit 2b44fcc

Browse files
author
DavidQ
committed
Reset /games/index.html to the new shared theme system and lock the updated shared-header rules.
Add Toolbox Aid as the first external menu item. Keep the shared header full width regardless of content. Preserve shared header sourcing under src/engine/theme. PR: BUILD_PR_STYLE_04_GAMES_INDEX_RESET_AND_HEADER_LOCKS
1 parent 98a737b commit 2b44fcc

8 files changed

Lines changed: 168 additions & 394 deletions

File tree

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
Header lock carry-forward notes
2+
3+
Locked for implementation:
4+
- first menu item = Toolbox Aid external link
5+
- external link standard = class is-external + target _blank + rel noopener noreferrer
6+
- header stretches full width regardless of content
7+
- no centered container may constrain the header
8+
- header image preserves aspect ratio
9+
10+
Execution slice:
11+
- reset /games/index.html only

docs/operations/dev/codex_commands.md

Lines changed: 15 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -2,35 +2,27 @@ MODEL: GPT-5.4-codex
22
REASONING: high
33

44
COMMAND:
5-
Create BUILD_PR_STYLE_02_INDEX_IMPLEMENTATION_AND_ROADMAP_EXPANSION as one focused, testable PR.
5+
Create BUILD_PR_STYLE_04_GAMES_INDEX_RESET_AND_HEADER_LOCKS as one focused, testable PR.
66

77
Rules:
88
- one PR purpose only
9-
- add to MASTER_ROADMAP_STYLE only; do not remove roadmap items
10-
- no big-bang migration
11-
- do not use existing HTML-JavaScript-Gaming styling as the base
12-
- derive the new style baseline from Toolbox Aid
9+
- use the established shared theme direction
10+
- do not use legacy project styling as the baseline
1311
- no embedded <style> in HTML
1412
- no inline style=""
1513
- no JS-generated styling
16-
- theme stays under src/engine/theme
17-
- keep HTML-page JS minimal
14+
- keep theme under src/engine/theme
1815

1916
Required work:
20-
1. Update docs/dev/roadmaps/MASTER_ROADMAP_STYLE.md with the bundled additive items.
21-
2. Create the theme foundation files under src/engine/theme:
22-
- main.css
23-
- header.css
24-
- nav.css
25-
- layout.css
26-
- pages.css
27-
- any additional theme file only if justified
28-
3. Pull the Toolbox Aid header into src/engine/theme/toolboxaid-header.html with only:
29-
- tagline changed to HTML-JavaScript Gaming
30-
- menu changed to Home / Games / Samples / Tools
31-
4. Extract the minimum required CSS so the header renders correctly without legacy dependencies.
32-
5. Reset /index.html onto the new theme and shared header.
33-
6. Remove old embedded/page-local styling dependencies from /index.html.
34-
7. Keep the implementation surgical and testable.
17+
1. Reset `/games/index.html` to the new shared style system.
18+
2. Use the shared header from `src/engine/theme/toolboxaid-header.html`.
19+
3. Update the shared header menu so the FIRST item is:
20+
<li class="menu-item">
21+
<a href="https://toolboxaid.com" class="is-external" target="_blank" rel="noopener noreferrer">Toolbox Aid</a>
22+
</li>
23+
4. Ensure header styling keeps the header full width regardless of content.
24+
5. Ensure the header image area preserves aspect ratio and is not constrained by a centered content wrapper.
25+
6. Add or keep shared external-link styling under theme CSS if useful.
26+
7. Keep `/games/index.html` structurally close to `/index.html` and `/samples/index.html`.
3527
8. Package to:
36-
<project folder>/tmp/BUILD_PR_STYLE_02_INDEX_IMPLEMENTATION_AND_ROADMAP_EXPANSION.zip
28+
<project folder>/tmp/BUILD_PR_STYLE_04_GAMES_INDEX_RESET_AND_HEADER_LOCKS.zip
Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
Expand MASTER_ROADMAP_STYLE with the missing style-system guardrails and implement the first real /index.html migration slice.
1+
Reset /games/index.html to the new shared theme system and lock the updated shared-header rules.
22

3-
Add the missing roadmap items so they are not forgotten.
4-
Reset /index.html onto the new Toolbox Aid-derived theme foundation.
5-
Keep the implementation narrow, testable, and free of embedded styling.
3+
Add Toolbox Aid as the first external menu item.
4+
Keep the shared header full width regardless of content.
5+
Preserve shared header sourcing under src/engine/theme.
66

7-
PR: BUILD_PR_STYLE_02_INDEX_IMPLEMENTATION_AND_ROADMAP_EXPANSION
7+
PR: BUILD_PR_STYLE_04_GAMES_INDEX_RESET_AND_HEADER_LOCKS
Lines changed: 53 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,53 @@
1+
# BUILD_PR_STYLE_04_GAMES_INDEX_RESET_AND_HEADER_LOCKS
2+
3+
## Purpose
4+
Apply the shared style system to `/games/index.html` and lock the newly agreed shared-header rules so they are implemented consistently.
5+
6+
## Single PR Purpose
7+
Reset `/games/index.html` onto the new Toolbox Aid-derived theme while carrying forward the agreed shared header behavior.
8+
9+
## Locked Header Rules For This PR
10+
1. Add this as the FIRST menu item:
11+
- Toolbox Aid
12+
- opens in a new tab
13+
- uses:
14+
- `class="is-external"`
15+
- `target="_blank"`
16+
- `rel="noopener noreferrer"`
17+
18+
2. Header must stretch the full width of the screen regardless of content.
19+
- no centered max-width wrapper may constrain the header
20+
- header image area must remain full-width
21+
- image behavior must preserve aspect ratio
22+
23+
3. Shared header source remains under:
24+
- `src/engine/theme/toolboxaid-header.html`
25+
26+
4. Shared header styling remains under:
27+
- `src/engine/theme/header.css`
28+
- plus shared base support in `src/engine/theme/main.css` as needed
29+
30+
## `/games/index.html` Scope
31+
- reset `/games/index.html` to shared theme only
32+
- use shared header
33+
- no embedded `<style>`
34+
- no inline `style=""`
35+
- no JS-generated styling
36+
- keep structure closely aligned with `/index.html` and `/samples/index.html`
37+
38+
## External Link Standard
39+
If multiple external links exist, standardize them using:
40+
- `class="is-external"`
41+
- `target="_blank"`
42+
- `rel="noopener noreferrer"`
43+
44+
Optional shared visual cue:
45+
- `is-external` may use a shared external-link marker in theme CSS
46+
47+
## Acceptance
48+
- `/games/index.html` visually aligns with the new shared page shell
49+
- header renders from shared source
50+
- Toolbox Aid link is first in menu and opens in new tab safely
51+
- header remains full width regardless of content
52+
- no embedded styling remains on `/games/index.html`
53+
- page is testable and narrow in scope

0 commit comments

Comments
 (0)