D25921: [libinput] Send touch events with respect to device rotation

Vlad Zahorodnii <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
zzag accepted this revision.
zzag added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> connection.cpp:254
> +#ifndef KWIN_BUILD_TESTING
> +QPointF devicePointToGlobalPosition(const QPointF &devicePos, const AbstractWaylandOutput *output)
> +{

Can be `static`.

> connection.cpp:259
> +    QPointF pos = devicePos;
> +    // TODO: Do we need to handle the flipped cases differently?
> +    switch (output->transform()) {

Yeah, I think so.

BRANCH
  touch-rotation-fix

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

To: romangg, #kwin, zzag
Cc: zzag, kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.