> On Jan. 7, 2014, 9:51 a.m., Martin Gräßlin wrote: > > how does that behave with the normal locker (that is no screensaver)? > > Thomas Lübking wrote: > "None" is not "blank" - the patch is correct. > > If a cursor is specified, it is displayed regardless of what window the > pointer is in. If None is specified, the normal cursor for that window is > displayed when the pointer is in grab_window or one of its subwindows; > otherwise, the cursor for grab_window is displayed. > --- http://tronche.com/gui/x/xlib/input/XGrabPointer.html > > Since the screenlocker qml window (nor the batterysucking fancy show) is > not the grab_window, both will initially get the default cursor of the > grab_window (atm. the left_arrow) and not receive mouse events (so w/ or w/o > the patch, the qml locker will not display an I-beam when hovering the > lineedit - at least not here, one would have to poll the mouse for this and > don't you dare* ;-) > > W/o an explicit grab_cursor, it however should be possible (never tested, > but seems the case given the patch) to alter the cursor shape from other > clients at any time (because it's not important for the grabbing client) > > * unless we make use of http://keithp.com/blogs/Cursor_tracking/ for PW/2
With the normal locker, I see no change in behaviour as well. The mouse cursor doesn't change, even when hovering over the password input field. If a screen saver is configured, the mouse cursor will change to the IBeam (i.e. text input) shape over the password field. But just to clarify: this is intended for the 4.11 branch only. Xscreensaver support is dropped for PW/2 anyway, right? - Wolfgang ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114841/#review46960 ----------------------------------------------------------- On Jan. 5, 2014, 9:55 a.m., Wolfgang Bauer wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/114841/ > ----------------------------------------------------------- > > (Updated Jan. 5, 2014, 9:55 a.m.) > > > Review request for kde-workspace and Martin Gräßlin. > > > Bugs: 311571 and 316459 > http://bugs.kde.org/show_bug.cgi?id=311571 > http://bugs.kde.org/show_bug.cgi?id=316459 > > > Repository: kde-workspace > > > Description > ------- > > Setting the cursor to ArrowCursor when calling XGrabPointer() prevents the > Screen savers from blanking the mouse cursor. > > I don't know why this has been done in the first place, but I couldn't see > any negative effect by setting it to None. Now the mouse cursor even changes > to the IBeam again when over the password field, which I find more intuitive. > > > Diffs > ----- > > ksmserver/screenlocker/ksldapp.cpp f0526cf > > Diff: https://git.reviewboard.kde.org/r/114841/diff/ > > > Testing > ------- > > Configure a Screen saver in systemsettings and wait for it to kick in (or > lock the screen manually). > Previously (since 4.10) the mouse cursor stayed visible, now it is blanked > like it was the case before 4.10. > Moving the mouse/pressing a key (to quit the Screen saver) makes the mouse > cursor appear again as it should, regardless of whether the screen is locked > or not. > > > Thanks, > > Wolfgang Bauer > >