Skip to content

Commit 8318cb5

Browse files
author
DavidQ
committed
Move Palette Manager V2 tag assignment into left Tags accordion and clean swatch labels - PR_26124_051-palette-manager-tags-accordion-and-label-cleanup
1 parent 6dbbd6e commit 8318cb5

15 files changed

Lines changed: 983 additions & 702 deletions

File tree

docs/dev/codex_commands.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,22 @@
1-
# Codex Commands - PR_26124_050-palette-manager-tag-entry-and-user-defined-visibility
1+
# Codex Commands - PR_26124_051-palette-manager-tags-accordion-and-label-cleanup
22

33
## Workflow
44
- Read `docs/dev/PROJECT_INSTRUCTIONS.md`.
55
- Used `.codex/skills/repo-build/SKILL.md`.
6-
- Created `docs/pr/PR_26124_050-palette-manager-tag-entry-and-user-defined-visibility/PLAN_PR.md`.
7-
- Created `docs/pr/PR_26124_050-palette-manager-tag-entry-and-user-defined-visibility/BUILD_PR.md`.
6+
- Created `docs/pr/PR_26124_051-palette-manager-tags-accordion-and-label-cleanup/PLAN_PR.md`.
7+
- Created `docs/pr/PR_26124_051-palette-manager-tags-accordion-and-label-cleanup/BUILD_PR.md`.
88
- Updated Palette Manager V2 only.
9-
- Created `docs/pr/PR_26124_050-palette-manager-tag-entry-and-user-defined-visibility/APPLY_PR.md`.
9+
- Created `docs/pr/PR_26124_051-palette-manager-tags-accordion-and-label-cleanup/APPLY_PR.md`.
1010

1111
## Validation Commands
1212
- `node --check tools/palette-manager-v2/controls/PaletteEditorControl.js`
1313
- `node --check tools/palette-manager-v2/modules/PaletteManagerApp.js`
14-
- Targeted served-browser Palette Manager V2 tag-entry/user-defined visibility check using Playwright from Node.
14+
- Targeted served-browser Palette Manager V2 Tags accordion and label cleanup check using Playwright from Node.
1515
- `git diff --check`
1616
- `git diff --cached --check`
1717
- `npm run test:workspace-v2`
1818
- `npm run codex:review-artifacts`
19-
- Python `zipfile` packaging for `tmp/PR_26124_050-palette-manager-tag-entry-and-user-defined-visibility_delta.zip`
19+
- Python `zipfile` packaging for `tmp/PR_26124_051-palette-manager-tags-accordion-and-label-cleanup_delta.zip`
2020

2121
## Validation Outcome
2222
- JavaScript syntax checks: PASS.

docs/dev/commit_comment.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Refine Palette Manager tag entry and user-defined visibility - PR_26124_050-palette-manager-tag-entry-and-user-defined-visibility
1+
Move Palette Manager tags into left accordion and clean labels - PR_26124_051-palette-manager-tags-accordion-and-label-cleanup
Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
# PR_26124_051 Report - Palette Manager Tags Accordion And Label Cleanup
2+
3+
## Summary
4+
- Removed duplicate Selected Swatch heading content.
5+
- Moved tag actions into one left-column Tags accordion.
6+
- Added click-to-toggle available tag buttons for the selected user swatch.
7+
- Compact field rows now reduce vertical space in Selected Swatch and Add.
8+
- Shortened Add/Update labels.
9+
10+
## Scope Check
11+
- Runtime code changed only under `tools/palette-manager-v2`.
12+
- No schema changes.
13+
- No `tools/shared` changes.
14+
- No sample JSON changes.
15+
- No workspace/toolState/session changes.
16+
- No dependency changes.
17+
18+
## Validation
19+
- `node --check tools/palette-manager-v2/controls/PaletteEditorControl.js`: PASS.
20+
- `node --check tools/palette-manager-v2/modules/PaletteManagerApp.js`: PASS.
21+
- Targeted served-browser Palette Manager V2 Tags accordion and label cleanup check: PASS.
22+
- `git diff --check`: PASS with Git LF-to-CRLF warnings for changed HTML/CSS files.
23+
- `git diff --cached --check`: PASS.
24+
- `npm run test:workspace-v2`: FAILED because the script is missing from `package.json`.
25+
- Full samples smoke test: skipped by instruction.
26+
27+
## Review Artifacts
28+
- `docs/dev/reports/codex_review.diff`
29+
- `docs/dev/reports/codex_changed_files.txt`
30+
- `tmp/PR_26124_051-palette-manager-tags-accordion-and-label-cleanup_delta.zip`

docs/dev/reports/codex_changed_files.txt

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -7,16 +7,16 @@ M docs/dev/codex_commands.md
77
M tools/palette-manager-v2/index.html
88
M tools/palette-manager-v2/modules/PaletteManagerApp.js
99
M tools/palette-manager-v2/paletteManagerV2.css
10-
?? docs/dev/reports/PR_26124_050_report.md
11-
?? docs/pr/PR_26124_050-palette-manager-tag-entry-and-user-defined-visibility/
10+
?? docs/dev/reports/PR_26124_051_report.md
11+
?? docs/pr/PR_26124_051-palette-manager-tags-accordion-and-label-cleanup/
1212

1313
# git diff --stat
14-
docs/dev/codex_commands.md | 21 +-
14+
docs/dev/codex_commands.md | 12 +-
1515
docs/dev/commit_comment.txt | 2 +-
1616
tools/palette-manager-v2/README.md | 6 +-
17-
.../controls/PaletteEditorControl.js | 240 +++++++++++++++++++--
18-
tools/palette-manager-v2/how_to_use.html | 8 +-
19-
tools/palette-manager-v2/index.html | 34 ++-
20-
.../modules/PaletteManagerApp.js | 66 ++++--
21-
tools/palette-manager-v2/paletteManagerV2.css | 61 ++++++
22-
8 files changed, 373 insertions(+), 65 deletions(-)
17+
.../controls/PaletteEditorControl.js | 193 +++++++--------------
18+
tools/palette-manager-v2/how_to_use.html | 12 +-
19+
tools/palette-manager-v2/index.html | 65 +++----
20+
.../modules/PaletteManagerApp.js | 94 ++++++++--
21+
tools/palette-manager-v2/paletteManagerV2.css | 24 +++
22+
8 files changed, 217 insertions(+), 191 deletions(-)

0 commit comments

Comments
 (0)