diff options
| author | Craig Jennings <c@cjennings.net> | 2026-06-16 05:11:27 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-06-16 05:11:27 -0500 |
| commit | 1a8f967d3bb0d3c74df9902252a2858a9449818d (patch) | |
| tree | 0899cd7188162aa1211d2695b68e36a4d03ce243 | |
| parent | afd2ddad818cdbf9f4b77d43efb91c35b6c57946 (diff) | |
| download | dotemacs-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.org | 5 |
1 files changed, 2 insertions, 3 deletions
@@ -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: |
