beischer added a comment.

  In D21884#481568 <https://phabricator.kde.org/D21884#481568>, @romangg wrote:
  
  > Ok, that would be even better. Only drawback is then that it shows the 
"wrong" interface in case libinput is used until a touchpad is plugged, but 
that's such an uncommon case that I don't think we should invest much more time 
into it. On the other side since libinput is the default backend in the future 
it might make more sense to show the libinput one per default if no touchpad is 
plugged in. Would this also be possible or is the current X libinput backend 
missing this funcitonality? At least from user perspective it could be 
confusing to see the non-default backend on the workstation without touchpad 
and something completely different on his laptop.
  
  
  I think one could set m_plugin to "TouchpadConfigLibinput(this, backend)" (as 
would be the case with a libinput touchpad). I just tried that, this produces 
the following result:
  
  F6908042: no_touchpad_libinput.png <https://phabricator.kde.org/F6908042>
  
  That doesn't look right to me? Maybe because 
TouchpadConfigContainer::kcmInit() is a no-op (first because the mode is Unset, 
so neither of the two if branches triggers, but even if that is fixed it's a 
noop because m_device is null in the xlibbackend).

REPOSITORY
  R119 Plasma Desktop

BRANCH
  bug-408325 (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D21884

To: beischer, ngraham, romangg, atulbi, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, 
ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart

Reply via email to