aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-06-16 05:11:27 -0500
committerCraig Jennings <c@cjennings.net>2026-06-16 05:11:27 -0500
commit1a8f967d3bb0d3c74df9902252a2858a9449818d (patch)
tree0899cd7188162aa1211d2695b68e36a4d03ce243
parentafd2ddad818cdbf9f4b77d43efb91c35b6c57946 (diff)
downloaddotemacs-1a8f967d3bb0d3c74df9902252a2858a9449818d.tar.gz
dotemacs-1a8f967d3bb0d3c74df9902252a2858a9449818d.zip
chore(todo): close the alphabetize-assignment-dropdown task
Resolved by afd2ddad. Dated-rewrote the VERIFY to a log entry per the todo format.
-rw-r--r--todo.org5
1 files changed, 2 insertions, 3 deletions
diff --git a/todo.org b/todo.org
index c193f1265..5deed3001 100644
--- a/todo.org
+++ b/todo.org
@@ -2813,9 +2813,8 @@ Fixed 2026-06-16: added pure helper =cj/gptel--model-to-symbol= (mirrors =cj/gpt
** VERIFY [#C] Dedup gptel model-switch commands — keep switch-backend or fold into change-model :bug:
=cj/gptel-change-model= (C-; a m) already does backend+model switching and interns correctly, so =cj/gptel-switch-backend= (C-; a B) is arguably redundant now that its crash is fixed. Decision for Craig: keep both, or delete =cj/gptel-switch-backend= plus its C-; a B binding and keep one model-switch command. From the 2026-06 config-audit follow-up.
-** VERIFY [#C] theme-studio: alphabetize the assignment view list below the package divider :feature:quick:solo:studio:next:
-Deferred from the batch (no blocker, needs a focused studio pass). Plan: sort the package-faces entries alphabetically below the divider in the assignment view dropdown/list. Localized to the view-list build; held only to keep the batch from sprawling.
-Sort the assignment view list alphabetically anywhere below the divider for the package views. From the roam inbox 2026-06-15.
+** 2026-06-16 Tue @ 05:10:55 -0500 Alphabetized the assignment-view package dropdown
+The package-faces optgroup (below the @code/@ui editor entries) now lists apps alphabetically by display label. Root cause: =buildViewSel= iterated =for(const app in APPS)=, and =generate.py= builds APPS as bespoke apps first then inventory apps, so the combined list wasn't alphabetical. Fix is localized to the view-list build per the plan: added a pure =appViewKeysSorted(apps)= helper in =app-core.js= (sorts keys by label, case-insensitive, key fallback when a label is missing) and =buildViewSel= iterates it. TDD: 4 node tests in =test-app-core.mjs= (red->green); updated the #viewtest browser gate from asserting insertion order to asserting =appViewKeysSorted(APPS)=; full theme-studio suite green (Python + Node + all browser gates). Commit =afd2ddad=, pushed. Visual sign-off optional (gate already confirms the DOM order).
** TODO [#C] theme-studio: calibre package doesn't color properly :bug:studio:
The calibre package preview has no elements to theme in the search list, and coloring switches to the string color on mismatched quotes. Investigate, then record a diagnosis and solution in this task before fixing. From the roam inbox 2026-06-15.
** VERIFY [#C] music: extract faces for music config :refactor:quick:solo:next: