LMMS/lmms

LCD Widgets reduce value by 2 when finishing trackpad "wheel scroll" action

Open

#3 370 ouverte le 19 févr. 2017

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)C++ (961 forks)batch import
buggood first issueguimacos

Métriques du dépôt

Stars
 (7 132 stars)
Métriques de merge PR
 (Merge moyen 76j 19h) (21 PRs mergées en 30 j)

Description

Here is a demonstration using the tempo control: lmms-extra-change-scroll-wheel-bug

The following events happen in the gif:

  1. When two fingers are placed first on the trackpad (the beginning of the wheel scroll motion) the tempo drops from 140 to 138;
  2. After a pause, the normal scrolling action changes the tempo to 128;
  3. The fingers are then lifted off the trackpad and the tempo mistakenly changes to 126.

More details on cause and suggested solution to follow.

Guide contributeur

Direction de recherche
Étudiez l'implémentation du widget LCD dans LMMS, probablement dans src/gui/widgets/LcdSpinBox.cpp ou similaire. Le bug se produit lors des événements de défilement de la molette du trackpad, où la valeur décrémente de 2 au début et à l'arrêt du défilement. Recherchez comment les événements de molette sont gérés et s'il y a un événement dupliqué ou un calcul incorrect du delta. Vérifiez s'il existe des différences dans la gestion des événements de molette de souris Qt entre une souris classique et un trackpad. Consultez les commentaires de l'issue #3370 pour toute indication des mainteneurs.
Stack technique
cpp
Domaine
desktop
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.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
C++Qt event handling
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40