romangg added inline comments.

INLINE COMMENTS

> libinputtouchpad.cpp:216
>      }
> +    KSharedConfigPtr configPtr = 
> KSharedConfig::openConfig(QStringLiteral("XlibinputConfigrc"));
> +    m_config = configPtr.data();

The name of the config file should be something similar to the other one and 
only lower case: `touchpadxlibinputrc`

> libinputtouchpad.cpp:426
> +        touchpadConfig.config()->sync();
> +    }
>      return true;

Why this write back? What's configProps.

REPOSITORY
  R119 Plasma Desktop

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

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

Reply via email to