diff options
| author | Craig Jennings <c@cjennings.net> | 2026-06-09 12:33:51 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-06-09 12:33:51 -0500 |
| commit | 59d4eea57e67bc82c9704a154458c2fc096a7331 (patch) | |
| tree | 04764516ee3b059c786d171c4ea46524ddaa485b /modules/ui-theme.el | |
| parent | 0ae74a6515fb21f09bccb6e6d4e0b68ca1878384 (diff) | |
| download | dotemacs-59d4eea57e67bc82c9704a154458c2fc096a7331.tar.gz dotemacs-59d4eea57e67bc82c9704a154458c2fc096a7331.zip | |
feat(theme-studio): add a real, exported :box face attribute
The mode-line box in the preview was hardcoded — it showed a box the generated theme couldn't actually produce, since build-theme.el never emitted :box. Made :box a real face attribute instead: a per-face box object (style line/raised/pressed, width, color) stored on UI and package faces, set from a "box" dropdown in both tables, rendered from the attribute everywhere (the mode-line bars, the package previews via ofs, the UI table preview cells), and exported through build-theme.el's --attrs as a proper :box plist (released/pressed → :style *-button; line → :line-width + optional :color). The hardcoded box is gone; mode-line and mode-line-inactive now default to the released-button box that is the Emacs default, so the preview and the export agree.
This also gives the package faces that genuinely use :box a way to represent it — the face audit found several (magit-branch-current/-remote-head, two flycheck list faces, the telega button family, ~15 slack button/dialog faces).
Tests: build-theme gains box-conversion + ui-box-emit ERT tests (24/24); the app-core deep-equal tests account for the new box slot; all 9 browser gates, 20 python, and 55 node tests stay green.
Diffstat (limited to 'modules/ui-theme.el')
0 files changed, 0 insertions, 0 deletions
