emacs-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

master 7a679953e2: Prevent pre-edit overlay text from being displayed af


From: Po Lu
Subject: master 7a679953e2: Prevent pre-edit overlay text from being displayed after a command
Date: Sat, 15 Jan 2022 05:21:24 -0500 (EST)

branch: master
commit 7a679953e22ada9df02b181b872b9e56bbfd0f07
Author: Po Lu <luangruo@yahoo.com>
Commit: Po Lu <luangruo@yahoo.com>

    Prevent pre-edit overlay text from being displayed after a command
    
    This works around buggy input methods causing the overlay to be
    displayed alongside newly inserted text for a brief period.
    
    * lisp/term/x-win.el (x-clear-preedit-text): New function.
    (x-preedit-text): Add said function to pre-command-hook.  It
    will remove itself when triggered.
---
 lisp/term/x-win.el | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/lisp/term/x-win.el b/lisp/term/x-win.el
index e52e488eda..cd63c9208b 100644
--- a/lisp/term/x-win.el
+++ b/lisp/term/x-win.el
@@ -1527,16 +1527,32 @@ This uses `icon-map-list' to map icon file names to 
stock icon names."
 (defvar x-preedit-overlay nil
   "The overlay currently used to display preedit text from a compose 
sequence.")
 
+;; With some input methods, text gets inserted before Emacs is told to
+;; remove any preedit text that was displayed, which causes both the
+;; preedit overlay and the text to be visible for a brief period of
+;; time.  This pre-command-hook clears the overlay before any command
+;; and should be set whenever a preedit overlay is visible.
+(defun x-clear-preedit-text ()
+  "Clear the pre-edit overlay and remove itself from pre-command-hook.
+This function should be installed in `pre-command-hook' whenever
+preedit text is displayed."
+  (when x-preedit-overlay
+    (delete-overlay x-preedit-overlay)
+    (setq x-preedit-overlay nil))
+  (remove-hook 'pre-command-hook #'x-clear-preedit-text))
+
 (defun x-preedit-text (event)
   "Display preedit text from a compose sequence in EVENT.
 EVENT is a preedit-text event."
   (interactive "e")
   (when x-preedit-overlay
     (delete-overlay x-preedit-overlay)
-    (setq x-preedit-overlay nil))
+    (setq x-preedit-overlay nil)
+    (remove-hook 'pre-command-hook #'x-clear-preedit-text))
   (when (nth 1 event)
     (let ((string (propertize (nth 1 event) 'face '(:underline t))))
       (setq x-preedit-overlay (make-overlay (point) (point)))
+      (add-hook 'pre-command-hook #'x-clear-preedit-text)
       (overlay-put x-preedit-overlay 'window (selected-window))
       (overlay-put x-preedit-overlay 'before-string
                    (if x-display-cursor-at-start-of-preedit-string



reply via email to

[Prev in Thread] Current Thread [Next in Thread]