Wrong return value of offset method for hidden/disconnected elements
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 42/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- jquery
- Domaine
- documentation
Piste de recherche
Commencez par la documentation liée de l’API offset, test/unit/offset.js et src/offset.js dans l’historique jQuery référencé. Confirmez la valeur de retour attendue pour les éléments masqués, avec display:none et déconnectés, puis rendez la documentation ou le comportement concernés cohérents et vérifiez que les tests de offset les couvrent.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
The jQuery docs state, that
While it is possible to get the coordinates of elements with visibility:hidden set, display:none is excluded from the rendering tree and thus has a position that is undefined.
The return value of undefined was changed a few years ago in https://github.com/jquery/jquery/pull/2396 due to https://github.com/jquery/jquery/issues/2310. The tests are here.
Now a value of { top: 0, left: 0 } is returned instead (https://github.com/jquery/jquery/blob/a684e6ba836f7c553968d7d026ed7941e1a612d8/src/offset.js#L88)
and undefined is only returned for an empty jQuery object (https://github.com/jquery/jquery/blob/a684e6ba836f7c553968d7d026ed7941e1a612d8/src/offset.js#L79).
- Langage dominant
- HTML
- Étoiles
- 324
- Forks
- 259
- Merge moyen
- 1 j 30 min
- PR mergées (30 j)
- 1
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
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 jquery/api.jquery.com
-
Mention of ‘timestamp’ in documentation for ‘cache’ parameter of ‘ajax’ is misleadingPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 62/100
jquery/api.jquery.com#1241 · 2 commentaires ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 68/100
jquery/api.jquery.com#1176 ·
-
The documentation of CSS methods should be further classifiedPeut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
jquery/api.jquery.com#1281 · 2 commentaires ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
jquery/api.jquery.com#1271 · 2 commentaires ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
jquery/api.jquery.com#1259 ·
Toutes les issues de jquery/api.jquery.com
Issues similaires
-
README says the land/sea mask must match the XS grid; the demo mask does not, and Step 4 worksOuvertedocumentation
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
buckai-observatory/Automated-Cloud-Removal-and-Seamless-Satellite-Image-Mosaicking-Pipeline#52 ·
Les mainteneurs répondent en général sous 1 jour
-
documentation
Difficulté 1/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 1-3 heures Accessibilité débutants 82/100
run-llama/llama_index#24163 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
btclib-org/btclib#3090 ·
Les mainteneurs répondent en général sous 1 jour
-
Translate the READMEOuvertedocumentation good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
asasemahmed/openboard#68 ·
Les mainteneurs répondent en général sous 1 jour