diff options
| author | Kévin Le Gouguec <kevin.legouguec@gmail.com> | 2023-05-06 09:36:25 +0200 |
|---|---|---|
| committer | Kévin Le Gouguec <kevin.legouguec@gmail.com> | 2023-05-06 09:39:12 +0200 |
| commit | aff208ffe98e7cf48339b1f415f7d18ea44da558 (patch) | |
| tree | 10f2da050a99e6df9b80cdd0022cee530118ee19 /itches/emacs/tracker.org | |
| parent | 1c6ad1435ce0453a8a7641461032b9fb5edd26e4 (diff) | |
| download | memory-leaks-aff208ffe98e7cf48339b1f415f7d18ea44da558.tar.xz | |
Shuffle more itches around
Diffstat (limited to 'itches/emacs/tracker.org')
| -rw-r--r-- | itches/emacs/tracker.org | 204 |
1 files changed, 103 insertions, 101 deletions
diff --git a/itches/emacs/tracker.org b/itches/emacs/tracker.org index 2240b69..a7e3f5b 100644 --- a/itches/emacs/tracker.org +++ b/itches/emacs/tracker.org @@ -158,83 +158,6 @@ Fixed by Andrea (see update 9 on his [[https://akrl.sdf.org/gccemacs.html][progr :Role: author :END: Unnoticed for a year, then merged with subsequent duplicate [[bug:33400]]. -*** TODO [[bug:31586]] ~frame-title-format~ doesn't save match data -:PROPERTIES: -:Role: watcher -:END: -*** DONE [[bug:33697]] file-truename messes with match data -:PROPERTIES: -:Role: author -:END: -Merged with [[bug:31586]], a more general issue dealing with match data -being "poisoned" by a user's =:eval= form in frame-title-format. -*** DONE [[bug:36803]] Update mode-line of every window when compilation ends -:PROPERTIES: -:Role: author -:END: -Fixed by Stefan. -*** DONE [[bug:57733]] Fix ~replace-*-in-region~ crash -:PROPERTIES: -:Role: author -:PatchApplied: t -:TestAdded: t -:END: -*** TODO Add command to toggle password visibility -**** cover letter -Hello Emacs, - -On occasion, e.g. when saving .authinfo.gpg, I'd like to briefly peek at -the password I'm typing. Entering the password twice is an appreciated -safeguard, but I know I'm perfectly able to make the same typo twice in -a row; I'd love for read-passwd to offer a command to toggle the -password visibility. - -(I'm dismissing the "write it in the clear, then cut it and paste it" -solution, since I don't want to add the kill ring nor my desktop -environment's clipboard manager to the equation) - -I'm attaching a proof-of-concept. Additional considerations, in no -particular order: - -- For the binding, I wanted something easy to mash quickly, but - something more mnemonic might be in order? - -- Some applications out there show the password while the keyboard - shortcut (or the mouse button) remains pressed, and hide it back as - soon as it is released; I like the idea, but I have no idea whether - that's feasible with Emacs. - -- The (member …) check could be replaced with e.g. (get-char-property - (minibuffer-prompt-end) 'display); that's how I implemented it at - first. It would fail to turn hiding off when the user has not entered - anything though. - -- I don't know how much documentation this new command would warrant: - - - Should it be mentioned in read-passwd's docstring? C-u isn't now, - but it was way back when, before 2012-04-11 "* lisp/subr.el - (read-passwd): Use read-string." (088be6fbd2). - - - Should it be mentioned in the manual? delete-minibuffer-contents - is documented of course, but not under "(elisp) Reading a - Password". - - - a NEWS entry, maybe? AFAICT C-u was never announced there. - -(On a side-note, which might warrant another bug report, I wonder if -anything could be done to advertise read-passwd-map bindings more. -Sure, C-h b shows these bindings, but users have no reason to assume -that read-passwd has extra bindings in addition to regular minibuffer -ones, so why would they reach for C-h b?) - -Let me know if this sounds like something useful. If so, and if the -implementation is acceptable as-is, I'll work on documentation as soon -as I have a clearer idea of where I should add some (and whether some -documentation should be added for C-u as well). - -Thank you for your time. - -TODO:[[file:read-passwd-reveal.patch][attach]]. *** TODO Make cursor appearance context-dependent E.g. add a customizable list of (PREDICATE . PROPERTIES) pairs; the cursor gets the PROPERTIES from the first PREDICATE which matches. @@ -242,6 +165,12 @@ cursor gets the PROPERTIES from the first PREDICATE which matches. Use-cases: - org-speed-commands - repeat-mode +*** compile +**** DONE [[bug:36803]] Update mode-line of every window when compilation ends +:PROPERTIES: +:Role: author +:END: +Fixed by Stefan. *** completion **** DONE [[bug:52169]] "Args out of range" when completing shell command :PROPERTIES: @@ -275,30 +204,6 @@ Cf. [[https://bugs.archlinux.org/task/58886][Arch Linux]], [[https://bugs.debian :Role: author :PatchApplied: t :END: -*** kill-ring -**** DONE [[bug:60841]] ~copy-region-blink-delay~ spuriously kicks in when the =region= face has =:inverse-video= -:PROPERTIES: -:Role: author -:PatchApplied: t -:END: -From ~emacs -Q~: -#+begin_src elisp -M-: (custom-set-faces '(region ((t (:foreground "blue" :inverse-video t))))) -C-x h -M-w -; ⇒ point moves to mark then back after copy-region-blink-delay seconds. -#+end_src - -This is caused by ~indicate-copied-region~ checking for =region= -having a =:background= to assess whether it's "highlighted". - -The bug report snowballed into - -- fixing a couple of corner-cases that made - ~face-differs-from-default-p~ less useful than it could be, -- adding a user-option for users who actually want that feedback - unconditionally, -- my first patch committed [[https://git.savannah.gnu.org/cgit/emacs.git/commit/?h=emacs-29&id=c4988840598b7da84b25d21a1936ce1ab6f6d666][in my own name]] 😲 *** imenu **** TODO Add grouping/sorting predicates to ~imenu~ completion So that ~imenu~ + ~icomplete-vertical-mode~ becomes a built-in, @@ -336,13 +241,110 @@ Simple example of these predicates in action: string pred)))) #+end_src +*** kill-ring +**** DONE [[bug:60841]] ~copy-region-blink-delay~ spuriously kicks in when the =region= face has =:inverse-video= +:PROPERTIES: +:Role: author +:PatchApplied: t +:END: +From ~emacs -Q~: +#+begin_src elisp +M-: (custom-set-faces '(region ((t (:foreground "blue" :inverse-video t))))) +C-x h +M-w +; ⇒ point moves to mark then back after copy-region-blink-delay seconds. +#+end_src + +This is caused by ~indicate-copied-region~ checking for =region= +having a =:background= to assess whether it's "highlighted". + +The bug report snowballed into + +- fixing a couple of corner-cases that made + ~face-differs-from-default-p~ less useful than it could be, +- adding a user-option for users who actually want that feedback + unconditionally, +- my first patch committed [[https://git.savannah.gnu.org/cgit/emacs.git/commit/?h=emacs-29&id=c4988840598b7da84b25d21a1936ce1ab6f6d666][in my own name]] 😲 +*** minibuffer +**** TODO Add command to toggle password visibility +***** cover letter +Hello Emacs, + +On occasion, e.g. when saving .authinfo.gpg, I'd like to briefly peek at +the password I'm typing. Entering the password twice is an appreciated +safeguard, but I know I'm perfectly able to make the same typo twice in +a row; I'd love for read-passwd to offer a command to toggle the +password visibility. + +(I'm dismissing the "write it in the clear, then cut it and paste it" +solution, since I don't want to add the kill ring nor my desktop +environment's clipboard manager to the equation) + +I'm attaching a proof-of-concept. Additional considerations, in no +particular order: + +- For the binding, I wanted something easy to mash quickly, but + something more mnemonic might be in order? + +- Some applications out there show the password while the keyboard + shortcut (or the mouse button) remains pressed, and hide it back as + soon as it is released; I like the idea, but I have no idea whether + that's feasible with Emacs. + +- The (member …) check could be replaced with e.g. (get-char-property + (minibuffer-prompt-end) 'display); that's how I implemented it at + first. It would fail to turn hiding off when the user has not entered + anything though. + +- I don't know how much documentation this new command would warrant: + + - Should it be mentioned in read-passwd's docstring? C-u isn't now, + but it was way back when, before 2012-04-11 "* lisp/subr.el + (read-passwd): Use read-string." (088be6fbd2). + + - Should it be mentioned in the manual? delete-minibuffer-contents + is documented of course, but not under "(elisp) Reading a + Password". + - a NEWS entry, maybe? AFAICT C-u was never announced there. + +(On a side-note, which might warrant another bug report, I wonder if +anything could be done to advertise read-passwd-map bindings more. +Sure, C-h b shows these bindings, but users have no reason to assume +that read-passwd has extra bindings in addition to regular minibuffer +ones, so why would they reach for C-h b?) + +Let me know if this sounds like something useful. If so, and if the +implementation is acceptable as-is, I'll work on documentation as soon +as I have a clearer idea of where I should add some (and whether some +documentation should be added for C-u as well). + +Thank you for your time. + +TODO:[[file:read-passwd-reveal.patch][attach]]. *** project **** DONE [[bug:45765]] Change default-directory before prompting in project-compile :PROPERTIES: :Role: author :PatchApplied: t :END: +*** regexp matching +**** TODO [[bug:31586]] ~frame-title-format~ doesn't save match data +:PROPERTIES: +:Role: watcher +:END: +**** DONE [[bug:33697]] file-truename messes with match data +:PROPERTIES: +:Role: author +:END: +Merged with [[bug:31586]], a more general issue dealing with match data +being "poisoned" by a user's =:eval= form in frame-title-format. +**** DONE [[bug:57733]] Fix ~replace-*-in-region~ crash +:PROPERTIES: +:Role: author +:PatchApplied: t +:TestAdded: t +:END: *** shr **** DONE [[bug:39504]] Ensure faces of enclosing elements apply to <code> elements :PROPERTIES: |
