Refine Palette Manager hover popover and source palette sort buttons - PR_26124_030-palette-hover-popover-and-sort-buttons

Author: DavidQ

docs/dev/reports/PR_26124_030_report.md

@@ -0,0 +1,19 @@
+# PR_26124_030 Palette Hover Popover And Sort Buttons
+
+## Summary
+- Converted source palette sort from dropdown to generated buttons: Original, Hue, Saturation, Brightness, Name.
+- Added active source sort button styling through `.is-active` and `aria-pressed`.
+- Kept User Palette sort as the existing dropdown.
+- Changed source swatch details back to an absolute popover positioned below the hovered/focused tile.
+- Kept the popover out of layout flow so it can overlap tiles below without shifting the grid.
+- Kept source palette data sourced from `src/engine/paletteList.js` only.
+
+## Validation
+- `node --check tools/palette-manager-v2/controls/SourcePaletteBrowserControl.js`
+- `node --check tools/palette-manager-v2/modules/PaletteManagerApp.js`
+- `node --check tools/palette-manager-v2/main.js`
+- `node --check tools/common/PaletteSortService.js`
+- `git diff --check` passed with line-ending warnings only.
+
+## Package
+- Delta ZIP: `tmp/PR_26124_030-palette-hover-popover-and-sort-buttons_delta.zip`

docs/dev/reports/codex_changed_files.txt

@@ -1,25 +1,17 @@
 # git status --short
 M docs/dev/reports/codex_review.diff
  M tools/palette-manager-v2/controls/SourcePaletteBrowserControl.js
- M tools/palette-manager-v2/controls/UserPaletteControl.js
  M tools/palette-manager-v2/index.html
- M tools/palette-manager-v2/main.js
  M tools/palette-manager-v2/modules/PaletteManagerApp.js
- M tools/palette-manager-v2/modules/SwatchRow.js
  M tools/palette-manager-v2/paletteManagerV2.css
-?? docs/dev/reports/PR_26124_029_report.md
-?? tools/common/PaletteSortService.js
+?? docs/dev/reports/PR_26124_030_report.md
 
 # git diff --stat (excluding codex review artifacts)
-.../controls/SourcePaletteBrowserControl.js        | 15 +++++++
- .../controls/UserPaletteControl.js                 | 20 ++++++++-
- tools/palette-manager-v2/index.html                |  8 ++++
- tools/palette-manager-v2/main.js                   |  2 +
- .../modules/PaletteManagerApp.js                   | 48 ++++++++++++++++++++--
- tools/palette-manager-v2/modules/SwatchRow.js      | 11 +++--
- tools/palette-manager-v2/paletteManagerV2.css      | 44 +++++++++++---------
- 7 files changed, 119 insertions(+), 29 deletions(-)
+.../controls/SourcePaletteBrowserControl.js        | 30 ++++++++++-----
+ tools/palette-manager-v2/index.html                |  8 ++--
+ .../modules/PaletteManagerApp.js                   |  2 +-
+ tools/palette-manager-v2/paletteManagerV2.css      | 45 +++++++++++++++++++++-
+ 4 files changed, 69 insertions(+), 16 deletions(-)
 
 # untracked file additions
- docs/dev/reports/PR_26124_029_report.md | 25 +
- tools/common/PaletteSortService.js | 121 +
+ docs/dev/reports/PR_26124_030_report.md | 19 +