D27975: Fix the tablet mode manager
Bhushan Shah <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
bshah created this revision. Herald added a project: KWin. Herald added a subscriber: kwin. bshah requested review of this revision. REVISION SUMMARY Previously we connected the hasTabletModeSwitchChanged to the tabletModeAvailableChanged which meant that in startup once libinput signaled us that we have switch, we would toggle that we now have a tablet mode. But, we were connecting this to wrong signal, as a result tablet mode would become available, but actual tablet switch inputspy was never getting registered, so tablet mode events were silently ignored. TEST PLAN tested on Dell Inspiron 7000 laptop where tablet mode switch is available REPOSITORY R108 KWin BRANCH bshah/fix-tablet-mode REVISION DETAIL https://phabricator.kde.org/D27975 AFFECTED FILES tabletmodemanager.cpp To: bshah Cc: kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart