Skip to content

Commit f604837

Browse files
author
DavidQ
committed
Add Palette Manager hover details layout and restore user/source palette sort controls - PR_26124_029-palette-hover-and-sort-controls
1 parent 3af8087 commit f604837

11 files changed

Lines changed: 803 additions & 102 deletions

File tree

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
# PR_26124_029 Palette Hover And Sort Controls
2+
3+
## Summary
4+
- Moved source swatch details out of the color tile overlay and into a below-tile details panel.
5+
- Hover/focus details now show name, symbol, hex, and source without overlapping neighboring tiles.
6+
- Added User Palette sort selector.
7+
- Added Source Palette sort selector.
8+
- Added reusable `tools/common/PaletteSortService.js` for original/source order, hue, saturation, brightness, and name sorting.
9+
- Sorting is view-only and does not mutate `tools.palette-browser.swatches` or read-only source palettes.
10+
11+
## Sort Note
12+
- Existing archive confirmed a prior palette sort set of Name, Hue, Saturation, and Lightness in `docs/archive/tools/SpriteEditor_old_keep/modules/controlSurfaceRightPanel.js`.
13+
- The new Brightness option uses the same HSL lightness metric while keeping the requested Name option.
14+
15+
## Validation
16+
- `node --check tools/common/PaletteSortService.js`
17+
- `node --check tools/palette-manager-v2/main.js`
18+
- `node --check tools/palette-manager-v2/modules/PaletteManagerApp.js`
19+
- `node --check tools/palette-manager-v2/modules/SwatchRow.js`
20+
- `node --check tools/palette-manager-v2/controls/UserPaletteControl.js`
21+
- `node --check tools/palette-manager-v2/controls/SourcePaletteBrowserControl.js`
22+
- Sort service smoke check confirmed original order returns unchanged ordering and name sort does not mutate the source array.
23+
24+
## Package
25+
- Delta ZIP: `tmp/PR_26124_029-palette-hover-and-sort-controls_delta.zip`
Lines changed: 20 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,25 @@
11
# git status --short
2-
AM docs/dev/reports/PR_26124_028_report.md
3-
M docs/dev/reports/codex_changed_files.txt
4-
MM docs/dev/reports/codex_review.diff
5-
MM src/engine/paletteList.js
6-
M tools/palette-manager-v2/controls/SourcePaletteBrowserControl.js
7-
M tools/palette-manager-v2/index.html
8-
M tools/palette-manager-v2/modules/PaletteManagerApp.js
9-
M tools/palette-manager-v2/modules/SwatchRow.js
10-
M tools/palette-manager-v2/paletteManagerV2.css
2+
M docs/dev/reports/codex_review.diff
3+
M tools/palette-manager-v2/controls/SourcePaletteBrowserControl.js
4+
M tools/palette-manager-v2/controls/UserPaletteControl.js
5+
M tools/palette-manager-v2/index.html
6+
M tools/palette-manager-v2/main.js
7+
M tools/palette-manager-v2/modules/PaletteManagerApp.js
8+
M tools/palette-manager-v2/modules/SwatchRow.js
9+
M tools/palette-manager-v2/paletteManagerV2.css
10+
?? docs/dev/reports/PR_26124_029_report.md
11+
?? tools/common/PaletteSortService.js
1112

1213
# git diff --stat (excluding codex review artifacts)
13-
docs/dev/reports/PR_26124_028_report.md | 4 ++--
14-
src/engine/paletteList.js | 26 ++++++++++++++++----------
15-
2 files changed, 18 insertions(+), 12 deletions(-)
14+
.../controls/SourcePaletteBrowserControl.js | 15 +++++++
15+
.../controls/UserPaletteControl.js | 20 ++++++++-
16+
tools/palette-manager-v2/index.html | 8 ++++
17+
tools/palette-manager-v2/main.js | 2 +
18+
.../modules/PaletteManagerApp.js | 48 ++++++++++++++++++++--
19+
tools/palette-manager-v2/modules/SwatchRow.js | 11 +++--
20+
tools/palette-manager-v2/paletteManagerV2.css | 44 +++++++++++---------
21+
7 files changed, 119 insertions(+), 29 deletions(-)
1622

1723
# untracked file additions
18-
(no output)
24+
docs/dev/reports/PR_26124_029_report.md | 25 +
25+
tools/common/PaletteSortService.js | 121 +

0 commit comments

Comments
 (0)