From: Christophe CURIS <[email protected]> A default cursor would make sense if we were talking about an API, where we would propose a 'I-don't-know' choice to devs, but here we're talking about WindowMaker's internal, where we're supposed to provide a consistent interface to the user, so picking a cursor is not an option (and 'Normal' is actually a decent looking choice anyway).
Suggested-by: Rodolfo García Peñas <[email protected]> --- src/WindowMaker.h | 5 +---- src/moveres.c | 6 +++--- src/screen.c | 2 +- src/wcore.c | 4 ++-- 4 files changed, 7 insertions(+), 10 deletions(-) diff --git a/src/WindowMaker.h b/src/WindowMaker.h index b5de45d..36d52f5 100644 --- a/src/WindowMaker.h +++ b/src/WindowMaker.h @@ -122,10 +122,7 @@ typedef enum { WCUR_EMPTY, /* Count of the number of cursors defined */ - WCUR_LAST, - - /* Alias for the Default cursor */ - WCUR_DEFAULT = WCUR_NORMAL + WCUR_LAST } w_cursor; /* geometry displays */ diff --git a/src/moveres.c b/src/moveres.c index 7e7cf5a..86cb88e 100644 --- a/src/moveres.c +++ b/src/moveres.c @@ -1240,7 +1240,7 @@ int wKeyboardMoveResizeWindow(WWindow * wwin) XGrabServer(dpy); XGrabPointer(dpy, scr->root_win, True, PointerMotionMask | ButtonReleaseMask | ButtonPressMask, GrabModeAsync, - GrabModeAsync, None, wPreferences.cursor[WCUR_DEFAULT], CurrentTime); + GrabModeAsync, None, wPreferences.cursor[WCUR_NORMAL], CurrentTime); @@ -2151,7 +2151,7 @@ void wSelectWindows(WScreen * scr, XEvent * ev) if (XGrabPointer(dpy, scr->root_win, False, ButtonMotionMask | ButtonReleaseMask | ButtonPressMask, GrabModeAsync, - GrabModeAsync, None, wPreferences.cursor[WCUR_DEFAULT], CurrentTime) != Success) { + GrabModeAsync, None, wPreferences.cursor[WCUR_NORMAL], CurrentTime) != Success) { return; } XGrabServer(dpy); @@ -2213,7 +2213,7 @@ void InteractivePlaceWindow(WWindow * wwin, int *x_ret, int *y_ret, unsigned wid int junk; if (XGrabPointer(dpy, root, True, PointerMotionMask | ButtonPressMask, - GrabModeAsync, GrabModeAsync, None, wPreferences.cursor[WCUR_DEFAULT], CurrentTime) != Success) { + GrabModeAsync, GrabModeAsync, None, wPreferences.cursor[WCUR_NORMAL], CurrentTime) != Success) { *x_ret = 0; *y_ret = 0; return; diff --git a/src/screen.c b/src/screen.c index 723ba0e..c296a33 100644 --- a/src/screen.c +++ b/src/screen.c @@ -460,7 +460,7 @@ static void createInternalWindows(WScreen * scr) attribs.override_redirect = True; attribs.background_pixmap = None; attribs.background_pixel = scr->white_pixel; - attribs.cursor = wPreferences.cursor[WCUR_DEFAULT]; + attribs.cursor = wPreferences.cursor[WCUR_NORMAL]; vmask |= CWColormap; attribs.colormap = scr->w_colormap; scr->dock_shadow = diff --git a/src/wcore.c b/src/wcore.c index e62b0fe..d04d093 100644 --- a/src/wcore.c +++ b/src/wcore.c @@ -47,7 +47,7 @@ WCoreWindow *wCoreCreateTopLevel(WScreen *screen, int x, int y, int width, int h vmask = CWBorderPixel | CWCursor | CWEventMask | CWOverrideRedirect | CWColormap; attribs.override_redirect = True; - attribs.cursor = wPreferences.cursor[WCUR_DEFAULT]; + attribs.cursor = wPreferences.cursor[WCUR_NORMAL]; attribs.background_pixmap = None; attribs.background_pixel = screen->black_pixel; attribs.border_pixel = border_pixel; @@ -98,7 +98,7 @@ WCoreWindow *wCoreCreate(WCoreWindow *parent, int x, int y, int width, int heigh core = wmalloc(sizeof(WCoreWindow)); vmask = CWBorderPixel | CWCursor | CWEventMask | CWColormap; - attribs.cursor = wPreferences.cursor[WCUR_DEFAULT]; + attribs.cursor = wPreferences.cursor[WCUR_NORMAL]; attribs.background_pixmap = None; attribs.background_pixel = parent->screen_ptr->black_pixel; attribs.event_mask = KeyPressMask | KeyReleaseMask | ButtonPressMask | -- 1.8.4.rc3 -- To unsubscribe, send mail to [email protected].
