airbnb/rheostat

RTL Support Doesn't Work

Open

#282 ouverte le 11 août 2020

Voir sur GitHub
 (6 commentaires) (0 réactions) (0 assignés)JavaScript (1 688 stars) (215 forks)batch import
bughelp wanted

Description

Thanks for your contributions and for supporting RTL!

We were testing rheostat slider and we encountered issues with RTL. (see gif below)

slider_rtl

Slider direction is flipped (opposite to the mouse movement direction)

relates to #235

Guide contributeur

Stack technique
javascriptreactcss
Domaine
frontendaccessibility
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-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
ReactCSS directionRTL concepts
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Investigate how the slider handles direction. Look at the component's handling of RTL and how the mouse movement maps to slider value. Review related issue #235 and the CSS direction properties. The bug may be due to incorrect calculation when dir='rtl'. Check the component's logic for updating position and consider mirroring the calculation for RTL.