mirror of
https://gitlab.winehq.org/wine/wine-staging.git
synced 2024-11-21 16:46:54 -08:00
231 lines
7.9 KiB
Diff
231 lines
7.9 KiB
Diff
From e7629ec320dcb35ad70e9dc31fbb0db5fb81cdc0 Mon Sep 17 00:00:00 2001
|
|
From: Felix Yan <felixonmars@gmail.com>
|
|
Date: Tue, 23 Sep 2014 23:22:17 +0800
|
|
Subject: [PATCH] winex11.drv: Update a candidate window's position with
|
|
over-the-spot style. (try 2)
|
|
|
|
In the current implementation, the candidate window position of a input
|
|
method is fixed because XNSpotLocation isn't updated after an input
|
|
context (XIC) is created in X11DRV_CreateIC().
|
|
X11DRV_UpdateCandidatePos() in this patch updates the position. You can
|
|
see the change of a position with ibus, scim or fcitx when input style
|
|
is set to "over the spot" in the registry key:
|
|
|
|
[HKEY_CURRENT_USER\Software\Wine\X11 Driver]
|
|
"InputStyle"="OverTheSpot"
|
|
|
|
This patch was based on the original work by Muneyuki Noguchi, and
|
|
received a lot of help from Sebastian Lackner.
|
|
---
|
|
dlls/user32/caret.c | 3 ++
|
|
dlls/win32u/driver.c | 7 ++++
|
|
dlls/winex11.drv/init.c | 1 +
|
|
dlls/winex11.drv/x11drv.h | 1 +
|
|
dlls/winex11.drv/xim.c | 72 ++++++++++++++++++++++++++++++++++++++-
|
|
include/wine/gdi_driver.h | 2 ++
|
|
6 files changed, 85 insertions(+), 1 deletion(-)
|
|
|
|
diff --git a/dlls/user32/caret.c b/dlls/user32/caret.c
|
|
index 8bf4962b708..33954123efe 100644
|
|
--- a/dlls/user32/caret.c
|
|
+++ b/dlls/user32/caret.c
|
|
@@ -27,6 +27,7 @@
|
|
#include "winbase.h"
|
|
#include "wingdi.h"
|
|
#include "ntuser.h"
|
|
+#include "user_private.h"
|
|
#include "wine/server.h"
|
|
#include "wine/debug.h"
|
|
|
|
@@ -274,6 +275,7 @@ BOOL WINAPI SetCaretPos( INT x, INT y )
|
|
r.left = x;
|
|
r.top = y;
|
|
CARET_DisplayCaret( hwnd, &r );
|
|
+ USER_Driver->pUpdateCandidatePos( hwnd, &r );
|
|
NtUserSetSystemTimer( hwnd, TIMERID, Caret.timeout, CARET_Callback );
|
|
}
|
|
return ret;
|
|
@@ -352,6 +354,7 @@ BOOL WINAPI ShowCaret( HWND hwnd )
|
|
if (ret && (hidden == 1)) /* hidden was 1 so it's now 0 */
|
|
{
|
|
CARET_DisplayCaret( hwnd, &r );
|
|
+ USER_Driver->pUpdateCandidatePos( hwnd, &r );
|
|
NtUserSetSystemTimer( hwnd, TIMERID, Caret.timeout, CARET_Callback );
|
|
}
|
|
return ret;
|
|
diff --git a/dlls/win32u/driver.c b/dlls/win32u/driver.c
|
|
index 6b6f1a64ce8..a073d05be27 100644
|
|
--- a/dlls/win32u/driver.c
|
|
+++ b/dlls/win32u/driver.c
|
|
@@ -924,6 +924,11 @@ static struct opengl_funcs *nulldrv_wine_get_wgl_driver( UINT version )
|
|
return (void *)-1;
|
|
}
|
|
|
|
+static void nulldrv_UpdateCandidatePos( HWND hwnd, const RECT *caret_rect )
|
|
+{
|
|
+
|
|
+}
|
|
+
|
|
static void nulldrv_ThreadDetach( void )
|
|
{
|
|
}
|
|
@@ -1218,6 +1223,7 @@ static const struct user_driver_funcs lazy_load_driver =
|
|
loaderdrv_wine_get_vulkan_driver,
|
|
/* opengl support */
|
|
nulldrv_wine_get_wgl_driver,
|
|
+ nulldrv_UpdateCandidatePos,
|
|
/* thread management */
|
|
nulldrv_ThreadDetach,
|
|
};
|
|
@@ -1287,6 +1293,7 @@ void CDECL __wine_set_user_driver( const struct user_driver_funcs *funcs, UINT v
|
|
SET_USER_FUNC(SystemParametersInfo);
|
|
SET_USER_FUNC(wine_get_vulkan_driver);
|
|
SET_USER_FUNC(wine_get_wgl_driver);
|
|
+ SET_USER_FUNC(UpdateCandidatePos);
|
|
SET_USER_FUNC(ThreadDetach);
|
|
#undef SET_USER_FUNC
|
|
|
|
diff --git a/dlls/winex11.drv/init.c b/dlls/winex11.drv/init.c
|
|
index 6f24e1bf370..5c4caed9b29 100644
|
|
--- a/dlls/winex11.drv/init.c
|
|
+++ b/dlls/winex11.drv/init.c
|
|
@@ -415,6 +415,7 @@ static const struct user_driver_funcs x11drv_funcs =
|
|
.pSystemParametersInfo = X11DRV_SystemParametersInfo,
|
|
.pwine_get_vulkan_driver = X11DRV_wine_get_vulkan_driver,
|
|
.pwine_get_wgl_driver = X11DRV_wine_get_wgl_driver,
|
|
+ .pUpdateCandidatePos = X11DRV_UpdateCandidatePos,
|
|
.pThreadDetach = X11DRV_ThreadDetach,
|
|
};
|
|
|
|
diff --git a/dlls/winex11.drv/x11drv.h b/dlls/winex11.drv/x11drv.h
|
|
index 2112980aab5..95a1dfdfa78 100644
|
|
--- a/dlls/winex11.drv/x11drv.h
|
|
+++ b/dlls/winex11.drv/x11drv.h
|
|
@@ -241,6 +241,7 @@ extern void X11DRV_WindowPosChanged( HWND hwnd, HWND insert_after, UINT swp_flag
|
|
struct window_surface *surface ) DECLSPEC_HIDDEN;
|
|
extern BOOL X11DRV_SystemParametersInfo( UINT action, UINT int_param, void *ptr_param,
|
|
UINT flags ) DECLSPEC_HIDDEN;
|
|
+extern void X11DRV_UpdateCandidatePos( HWND hwnd, const RECT *caret_rect ) DECLSPEC_HIDDEN;
|
|
extern void X11DRV_ThreadDetach(void) DECLSPEC_HIDDEN;
|
|
|
|
/* X11 driver internal functions */
|
|
diff --git a/dlls/winex11.drv/xim.c b/dlls/winex11.drv/xim.c
|
|
index 6adf2475de7..7a574e41e27 100644
|
|
--- a/dlls/winex11.drv/xim.c
|
|
+++ b/dlls/winex11.drv/xim.c
|
|
@@ -30,6 +30,7 @@
|
|
#include "x11drv.h"
|
|
#include "imm.h"
|
|
#include "wine/debug.h"
|
|
+#include "wine/server.h"
|
|
#include "wine/unicode.h"
|
|
|
|
WINE_DEFAULT_DEBUG_CHANNEL(xim);
|
|
@@ -461,6 +462,48 @@ static BOOL X11DRV_DestroyIC(XIC xic, XPointer p, XPointer data)
|
|
return TRUE;
|
|
}
|
|
|
|
+/***********************************************************************
|
|
+ * X11DRV_UpdateCandidatePos
|
|
+ */
|
|
+void X11DRV_UpdateCandidatePos( HWND hwnd, const RECT *caret_rect )
|
|
+{
|
|
+ if (ximStyle & XIMPreeditPosition)
|
|
+ {
|
|
+ struct x11drv_win_data *data;
|
|
+ HWND parent;
|
|
+
|
|
+ for (parent = hwnd; parent && parent != GetDesktopWindow(); parent = GetAncestor( parent, GA_PARENT ))
|
|
+ {
|
|
+ if (!(data = get_win_data( parent ))) continue;
|
|
+ if (data->xic)
|
|
+ {
|
|
+ XVaNestedList preedit;
|
|
+ XPoint xpoint;
|
|
+ POINT pt;
|
|
+
|
|
+ pt.x = caret_rect->left;
|
|
+ pt.y = caret_rect->bottom;
|
|
+
|
|
+ if (hwnd != data->hwnd)
|
|
+ MapWindowPoints( hwnd, data->hwnd, &pt, 1 );
|
|
+
|
|
+ if (GetWindowLongW( data->hwnd, GWL_EXSTYLE ) & WS_EX_LAYOUTRTL)
|
|
+ pt.x = data->client_rect.right - data->client_rect.left - 1 - pt.x;
|
|
+
|
|
+ xpoint.x = pt.x + data->client_rect.left - data->whole_rect.left;
|
|
+ xpoint.y = pt.y + data->client_rect.top - data->whole_rect.top;
|
|
+
|
|
+ preedit = XVaCreateNestedList( 0, XNSpotLocation, &xpoint, NULL );
|
|
+ if (preedit)
|
|
+ {
|
|
+ XSetICValues( data->xic, XNPreeditAttributes, preedit, NULL );
|
|
+ XFree( preedit );
|
|
+ }
|
|
+ }
|
|
+ release_win_data( data );
|
|
+ }
|
|
+ }
|
|
+}
|
|
|
|
XIC X11DRV_CreateIC(XIM xim, struct x11drv_win_data *data)
|
|
{
|
|
@@ -488,7 +531,7 @@ XIC X11DRV_CreateIC(XIM xim, struct x11drv_win_data *data)
|
|
XNDestroyCallback, &destroy,
|
|
NULL);
|
|
data->xic = xic;
|
|
- return xic;
|
|
+ goto return_xic;
|
|
}
|
|
|
|
/* create callbacks */
|
|
@@ -586,5 +629,32 @@ XIC X11DRV_CreateIC(XIM xim, struct x11drv_win_data *data)
|
|
if (status != NULL)
|
|
XFree(status);
|
|
|
|
+return_xic:
|
|
+ if (xic != NULL && (ximStyle & XIMPreeditPosition))
|
|
+ {
|
|
+ SERVER_START_REQ( set_caret_info )
|
|
+ {
|
|
+ req->flags = 0; /* don't set anything */
|
|
+ req->handle = 0;
|
|
+ req->x = 0;
|
|
+ req->y = 0;
|
|
+ req->hide = 0;
|
|
+ req->state = 0;
|
|
+ if (!wine_server_call_err( req ))
|
|
+ {
|
|
+ HWND hwnd;
|
|
+ RECT r;
|
|
+
|
|
+ hwnd = wine_server_ptr_handle( reply->full_handle );
|
|
+ r.left = reply->old_rect.left;
|
|
+ r.top = reply->old_rect.top;
|
|
+ r.right = reply->old_rect.right;
|
|
+ r.bottom = reply->old_rect.bottom;
|
|
+ X11DRV_UpdateCandidatePos( hwnd, &r );
|
|
+ }
|
|
+ }
|
|
+ SERVER_END_REQ;
|
|
+ }
|
|
+
|
|
return xic;
|
|
}
|
|
diff --git a/include/wine/gdi_driver.h b/include/wine/gdi_driver.h
|
|
index a63e2c3ceda..79e9f869b6e 100644
|
|
--- a/include/wine/gdi_driver.h
|
|
+++ b/include/wine/gdi_driver.h
|
|
@@ -324,6 +324,8 @@ struct user_driver_funcs
|
|
const struct vulkan_funcs * (*pwine_get_vulkan_driver)(UINT);
|
|
/* opengl support */
|
|
struct opengl_funcs * (*pwine_get_wgl_driver)(UINT);
|
|
+ /* IME functions */
|
|
+ void (*pUpdateCandidatePos)(HWND, const RECT *);
|
|
/* thread management */
|
|
void (*pThreadDetach)(void);
|
|
};
|
|
--
|
|
2.34.1
|
|
|