yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[effects] Use standard arrow cursor in Present Windows & Desktop Grid

Browse source 
The pointing finger cursor is used to show that a piece of text is a
clickable link, and it is inappropriate to use it in other contexts.
Regular UI elements that do something when single-clicked continue to
use the standard arrow cursor in other contexts, so let's follow that
convention in the Present Windows and Desktop Grid effects too.

BUG: 421928
FIXED-IN: 5.23
This commit is contained in:
Nate Graham 2021-06-17 14:01:48 -06:00
parent ca3609eb4d
commit c9a719e109
2 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/effects/desktopgrid/desktopgrid.cpp
View file

@ -610,7 +610,7 @@ void DesktopGridEffect::windowInputMouseEvent(QEvent* e)
}
}
if (wasWindowMove || wasDesktopMove) { // reset pointer
effects->defineCursor(Qt::PointingHandCursor);
effects->defineCursor(Qt::ArrowCursor);
} else { // click -> exit
const int desk = posToDesktop(me->pos());
if (desk > effects->numberOfDesktops())
@ -1036,7 +1036,7 @@ void DesktopGridEffect::setup()
return;
if (!keyboardGrab) {
keyboardGrab = effects->grabKeyboard(this);
effects->startMouseInterception(this, Qt::PointingHandCursor);
effects->startMouseInterception(this, Qt::ArrowCursor);
effects->setActiveFullScreenEffect(this);
}
setHighlightedDesktop(effects->currentDesktop());

2
src/effects/presentwindows/presentwindows.cpp
View file

@ -1650,7 +1650,7 @@ void PresentWindowsEffect::setActive(bool active)
}
// Create temporary input window to catch mouse events
effects->startMouseInterception(this, Qt::PointingHandCursor);
effects->startMouseInterception(this, Qt::ArrowCursor);
m_hasKeyboardGrab = effects->grabKeyboard(this);
effects->setActiveFullScreenEffect(this);