GNU bug report logs -
#47377
[PATCH] Fix preeditarea reporting wrong spot
Previous Next
Reported by: Amos Bird <amosbird <at> gmail.com>
Date: Thu, 25 Mar 2021 06:56:02 UTC
Severity: normal
Tags: patch
Done: Eli Zaretskii <eliz <at> gnu.org>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 47377 in the body.
You can then email your comments to 47377 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
bug-gnu-emacs <at> gnu.org
:
bug#47377
; Package
emacs
.
(Thu, 25 Mar 2021 06:56:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Amos Bird <amosbird <at> gmail.com>
:
New bug report received and forwarded. Copy sent to
bug-gnu-emacs <at> gnu.org
.
(Thu, 25 Mar 2021 06:56:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
[Message part 1 (text/plain, inline)]
This patch adjust the x position of preeditarea with both left
fringe
and left margin, which prevents IME preedit box (such as fcitx)
from
placing at the wrong position in GUI emacs.
This fixes the bug described in
https://lists.defectivebydesign.org/archive/html/emacs-devel/2017-02/msg00817.html
I've already assigned the form for contributing to emacs.
regards,
Amos
[0001-Fix-preeditarea-reporting-wrong-spot.patch (text/x-patch, inline)]
From 769adf8858c067e779c07087f53048455ff1c7f3 Mon Sep 17 00:00:00 2001
From: Amos Bird <amosbird <at> gmail.com>
Date: Thu, 25 Mar 2021 14:50:46 +0800
Subject: [PATCH] Fix preeditarea reporting wrong spot.
This patch adjust the x position of preeditarea with both left fringe
and left margin, which prevents IME preedit box (such as fcitx) from
placing at the wrong position in GUI emacs.
---
src/xfns.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/xfns.c b/src/xfns.c
index d90644819b..0507dc8f61 100644
--- a/src/xfns.c
+++ b/src/xfns.c
@@ -2783,7 +2783,7 @@ xic_set_preeditarea (struct window *w, int x, int y)
XVaNestedList attr;
XPoint spot;
- spot.x = WINDOW_TO_FRAME_PIXEL_X (w, x) + WINDOW_LEFT_FRINGE_WIDTH (w);
+ spot.x = WINDOW_TO_FRAME_PIXEL_X (w, x) + WINDOW_LEFT_FRINGE_WIDTH (w) + WINDOW_LEFT_MARGIN_WIDTH(w);
spot.y = WINDOW_TO_FRAME_PIXEL_Y (w, y) + FONT_BASE (FRAME_FONT (f));
attr = XVaCreateNestedList (0, XNSpotLocation, &spot, NULL);
XSetICValues (FRAME_XIC (f), XNPreeditAttributes, attr, NULL);
--
2.31.0
Reply sent
to
Eli Zaretskii <eliz <at> gnu.org>
:
You have taken responsibility.
(Thu, 25 Mar 2021 15:27:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
Amos Bird <amosbird <at> gmail.com>
:
bug acknowledged by developer.
(Thu, 25 Mar 2021 15:27:02 GMT)
Full text and
rfc822 format available.
Message #10 received at 47377-done <at> debbugs.gnu.org (full text, mbox):
> From: Amos Bird <amosbird <at> gmail.com>
> Date: Thu, 25 Mar 2021 14:57:13 +0800
>
> This patch adjust the x position of preeditarea with both left
> fringe
> and left margin, which prevents IME preedit box (such as fcitx)
> from
> placing at the wrong position in GUI emacs.
>
> This fixes the bug described in
> https://lists.defectivebydesign.org/archive/html/emacs-devel/2017-02/msg00817.html
>
> I've already assigned the form for contributing to emacs.
Thanks, installed.
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Fri, 23 Apr 2021 11:24:12 GMT)
Full text and
rfc822 format available.
This bug report was last modified 3 years and 2 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.