Remove enableUserSelectHack, set to false by default
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 45/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- javascript
- Domaine
- frontend
Piste de recherche
Start with lib/DraggableCore.js around line 97 and lib/utils/domFns.js around line 168 to understand how enableUserSelectHack is defined and consumed. Done means the hack is removed or defaults to false without clearing the user's document selection; verify the Gmail compose reproduction described in the issue.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Rant: We had an incredibly difficult time connecting the dots between a setState causing a modal to hide and figuring out why the user was losing focus in Gmail. I've probably spent 10+ hours digging through our codebase since we have a ton of places where we manage the user's selection. The last thing I expected was for the Draggable wrapper on that modal to be removing all ranges from the document. I had to resort to a binary search "comment half of everything out, see if it still works, repeat" which was still extremely difficult because I didn't consider the very last element to be the one causing issues.
Repro: The user selects text in any Gmail compose window, opens formatting, clicks bold, our modal disappears (from something like document.addEventListener('mousedown', checkIsOutsideAndHideModal), and the user loses their selection and bold is not applied
RCA:
https://github.com/react-grid-layout/react-draggable/blob/master/lib/DraggableCore.js#L97
This comment explains why it exists, but the code that consumes this flag is mostly about removing document ranges
https://github.com/react-grid-layout/react-draggable/blob/master/lib/utils/domFns.js#L168
- Langage dominant
- JavaScript
- Étoiles
- 9.3k
- Forks
- 1k
- Merge moyen
- 2 h 35 min
- PR mergées (30 j)
- 4
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de react-grid-layout/react-draggable
-
React.lazy(() => import('react-draggable')) fails TypeScript: typeof Draggable is not assignable to ComponentTypePeut-être pris @tevinch l’a pris il y a 15 jours. Ouvertebug ready-for-agent
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
react-grid-layout/react-draggable#822 · 2 commentaires ·
-
V4.5.0 Chrome Extension ErrorOuvertebug needs-info
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
react-grid-layout/react-draggable#782 · 2 commentaires ·
-
bug needs-info
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
react-grid-layout/react-draggable#772 · 1 commentaire ·
-
error in react 19Peut-être pris @Juice-de-Orange l’a pris il y a 11 jours. Ouvertebug ready-for-agent
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
react-grid-layout/react-draggable#771 · 14 commentaires · 31 réactions ·
-
enhancement ready-for-agent
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
react-grid-layout/react-draggable#762 · 1 commentaire ·
Toutes les issues de react-grid-layout/react-draggable
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
jfromaniello/url-join#134 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 60/100
quarto-dev/quarto-cli#15042 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
extendr/extendr.github.io#134 ·