diff options
| -rw-r--r-- | docs/NOTES.org | 39 | ||||
| -rw-r--r-- | modules/modeline-config.el | 3 | ||||
| -rw-r--r-- | modules/weather-config.el | 25 |
3 files changed, 64 insertions, 3 deletions
diff --git a/docs/NOTES.org b/docs/NOTES.org index 30243dc5..7de74826 100644 --- a/docs/NOTES.org +++ b/docs/NOTES.org @@ -484,7 +484,44 @@ If Craig or Claude need more context: ** 🚀 Current Session Notes -*** 2025-11-04 Session - Complete Transcription Workflow Implementation +*** 2025-11-04 Session 2 - Emergency Bug Fixes & Modeline Polish +*Time:* ~30 minutes +*Status:* ✅ COMPLETE - Fixed async buffer error and improved modeline spacing + +*What We Completed:* + +1. ✅ **Fixed Critical Async Buffer Error on Startup** + - Problem: "Selecting deleted buffer" error in async.el during init + - Root cause: wttrin-mode-line-mode starting async HTTP request during init + - Solution: Delayed activation using after-init-hook + - Weather widget now loads AFTER Emacs finishes initializing + - Prevents async buffer from being killed before request completes + +2. ✅ **Fixed Parenthesis Error in weather-config.el** + - Removed extra closing parenthesis that prevented file from loading + - Verified with check-parens + +3. ✅ **Improved Modeline Right-Side Spacing** + - Problem: Right-most icon (weather) appeared flush with window edge + - Discovered: Regular spaces were being trimmed by right-align mechanism + - Solution: Added 2 non-breaking spaces (U+00A0) after misc-info segment + - Provides visual breathing room without being excessive + +*Key Technical Insights:* +- Async operations during init need careful timing (use after-init-hook) +- mode-line-right-align-edge 'right-margin trims trailing regular spaces +- Non-breaking spaces (U+00A0) survive trimming in modeline format + +*Files Modified:* +- modules/weather-config.el - Added after-init-hook delay for mode-line widget +- modules/modeline-config.el - Added 2 non-breaking spaces at end + +*Status:* +- Emacs launches without errors ✅ +- Weather widget appears in modeline after startup ✅ +- Modeline spacing looks clean and professional ✅ + +*** 2025-11-04 Session 1 - Complete Transcription Workflow Implementation *Time:* ~3 hours *Status:* ✅ COMPLETE - Full async transcription system with 60 passing tests diff --git a/modules/modeline-config.el b/modules/modeline-config.el index 140d21cd..b1403539 100644 --- a/modules/modeline-config.el +++ b/modules/modeline-config.el @@ -156,7 +156,8 @@ Shows only in active window.") mode-line-format-right-align cj/modeline-vc-branch " " - cj/modeline-misc-info)) + cj/modeline-misc-info + " ")) ;; Mark all segments as risky-local-variable (required for :eval forms) (dolist (construct '(cj/modeline-buffer-name diff --git a/modules/weather-config.el b/modules/weather-config.el index 31fb1b70..55eddf16 100644 --- a/modules/weather-config.el +++ b/modules/weather-config.el @@ -10,8 +10,15 @@ ;; ----------------------------------- Wttrin ---------------------------------- +;; Load wttrin from local development directory +(add-to-list 'load-path "/home/cjennings/code/wttrin") + +;; Set debug flag BEFORE loading wttrin (checked at load time) +(setq wttrin-debug t) + (use-package wttrin - :vc (:url "https://github.com/cjennings/emacs-wttrin" :rev :newest) + ;; Uncomment the next line to use vc-install instead of local directory: + ;; :vc (:url "https://github.com/cjennings/emacs-wttrin" :rev :newest) :defer t :preface ;; dependency for wttrin @@ -21,6 +28,22 @@ ("M-W" . wttrin) :custom (wttrin-unit-system "u") + (wttrin-mode-line-favorite-location "New Orleans, LA") + (wttrin-mode-line-refresh-interval 900) ; 15 minutes + :init + ;; Explicitly autoload the mode function (needed for local dev directory) + (autoload 'wttrin-mode-line-mode "wttrin" "Toggle weather display in mode-line." t) + ;; Enable mode-line widget AFTER Emacs finishes initializing + ;; (url-retrieve async needs full init to work without buffer errors) + (if (daemonp) + ;; Daemon mode: wait for first client to connect + (add-hook 'server-after-make-frame-hook + (lambda () (wttrin-mode-line-mode 1)) + t) ; append to end of hook + ;; Normal Emacs: wait for startup to complete + (add-hook 'after-init-hook + (lambda () (wttrin-mode-line-mode 1)) + t)) ; append to end of hook :config (setq wttrin-default-locations '( "New Orleans, LA" |