qmk/qmk_firmware

[Bug] Cirque trackpad does not consider rotation value for scroll and secondary click gestures in relative mode

Open

#21 010 ouverte le 21 mai 2023

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)C (43 867 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (20 368 stars)
Métriques de merge PR
 (Merge moyen 20j 9h) (27 PRs mergées en 30 j)

Description

Describe the Bug

Hi,

I am currently using cirque trackpad on my custom keyboard with following settings:

#define CIRQUE_PINNACLE_ADDR 0x2A

#define CIRQUE_PINNACLE_CURVED_OVERLAY

#define CIRQUE_PINNACLE_ATTENUATION EXTREG__TRACK_ADCCONFIG__ADC_ATTENUATE_1X

#define POINTING_DEVICE_ROTATION_90

#define CIRQUE_PINNACLE_POSITION_MODE CIRQUE_PINNACLE_RELATIVE_MODE 

#define CIRQUE_PINNACLE_TAP_ENABLE

#define CIRQUE_PINNACLE_SECONDARY_TAP_ENABLE

#define POINTING_DEVICE_GESTURES_SCROLL_ENABLE

The expected behaviour of the trackpad for scroll is to initiate from the right of the trackpad, but on the rotated device its on the top (physical right) and same is for secondary click, it should be triggered from the top right , but due to 90 deg roation, its triggered from top left.

Keyboard Used

No response

Link to product page (if applicable)

No response

Operating System

windows

qmk doctor Output

Ψ QMK Doctor is checking your environment.
Ψ CLI version: 1.1.1
Ψ QMK home: C:/Users/harsh/qmk_firmware
Ψ Detected Windows 10 (10.0.22621).
Ψ QMK MSYS version: 1.7.2
Ψ Git branch: p36_unibody
Ψ Repo version: 0.20.5

Is AutoHotKey / Karabiner installed

  • AutoHotKey (Windows)
  • Karabiner (macOS)

Other keyboard-related software installed

No response

Additional Context

No response

Guide contributeur

Direction de recherche
Examinez les fichiers du pilote Cirque Pinnacle, en particulier `drivers/sensors/cirque pinnacle.c` et `drivers/sensors/cirque pinnacle.h`. Recherchez comment `POINTING DEVICE ROTATION 90` est appliqué aux coordonnées des gestes (défilement et clic secondaire). Le bug indique que la rotation est appliquée au mouvement du curseur mais pas aux gestes. Vérifiez les fonctions de traitement des gestes comme `cirque pinnacle gesture tap` et `cirque pinnacle gesture scroll`. Vérifiez si la transformation de rotation y est absente. Un correctif impliquerait de faire pivoter les coordonnées tactiles avant la détection des gestes. Testez avec un pavé tactile Cirque en mode relatif avec rotation activée pour confirmer le correctif.
Stack technique
c
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65