Update the documentation of jQuery.data to indicate what the function actually accepts for the 1st parameter.
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 52/100
- Type d'issue
- Documentation
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- jquery
- Domaine
- documentation
Piste de recherche
Commencez par la page de documentation de jQuery.data et comparez ses signatures de paramètres avec src/data/var/acceptData.js, que l’issue identifie comme preuve des nœuds pris en charge. Mettez à jour la documentation afin que le premier paramètre reflète correctement le type Document accepté et clarifiez le comportement de l’objet mentionné si cela est confirmé. C’est terminé lorsque les signatures publiées correspondent aux entrées prises en charge.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
The documentation for jQuery.data gives the following signatures:
jQuery.data( element, key, value )
jQuery.data( element, key )
jQuery.data( element )
In all three the element parameter is typed as Element. However, jQuery does accept an element which is of type Document. Two pieces of evidence:
-
In practice, I've used
jQuery.dataonDocumentobjects for years, without any trouble. -
The data machinery calls on
acceptDatato detect whether a node is fit to be used as the first argument of thedatafunction.acceptData's code and comments both explicitly consider thatDocumentis a valid node.
It seems that JavaScript objects that do not have a numeric nodeType field can also be used with jQuery.data. I'm mentioning this for completeness. I've never used this capability.
- 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