Specify that .data and .attr can't be mixed
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Documentación
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- jquery
- Área
- documentation
Línea de trabajo
Comienza con la documentación de .attr y .data y revisa el ejemplo de CodePen enlazado para confirmar el comportamiento descrito. Se considera terminado cuando la documentación pertinente advierte claramente que mezclar .data y .attr puede producir resultados inesperados porque no actualizan el mismo estado del DOM.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I found out the hard way that you can't mix usage of .data and .attr.
The .data documentation clearly states that it doesn't change the DOM upon setting a value.
However, the .attr documentation talks at length about the difference between it and prop, but does not state that it doesn't change the dataset of the DOM node, as demonstrated here: https://codepen.io/cosmo0/pen/YzKVBPp?editors=0011
So your code has to be very strict about using either .data or .attr, because mixing both can result in unexpected behavior, at least when you don't know how jQuery works.
I understand it's by-design, but maybe a warning in the .attr documentation (and/or the .data?) would help people to not fall for this trap?
- Lenguaje dominante
- HTML
- Estrellas
- 324
- Forks
- 259
- Merge medio
- 1 d 30 min
- PR fusionados (30 d)
- 1
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de jquery/api.jquery.com
-
Mention of ‘timestamp’ in documentation for ‘cache’ parameter of ‘ajax’ is misleadingPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
jquery/api.jquery.com#1241 · 2 comentarios ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
jquery/api.jquery.com#1176 ·
-
The documentation of CSS methods should be further classifiedQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
jquery/api.jquery.com#1281 · 2 comentarios ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
jquery/api.jquery.com#1271 · 2 comentarios ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
jquery/api.jquery.com#1259 ·
Todos los issues de jquery/api.jquery.com
Issues similares
-
5-minutes algorithm all-levels any-language beginner-friendly community documentation easy first-timers-only good first issue hacktoberfest help wanted learning no-code-needed open-source python students
Dificultad 2/5 Menos de una hora Aptitud para principiantes 88/100
Huerte/algo-solutions-python#253 ·
Los mantenedores suelen responder en 1 día
-
size/s
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
sipyourdrink-ltd/bernstein#7179 ·
Los mantenedores suelen responder en 1 día
-
area/docs
Dificultad 1/5 Menos de una hora Aptitud para principiantes 86/100
meshery/meshery#23375 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Synthetic
Dificultad 1/5 Menos de una hora Aptitud para principiantes 66/100
samtoth/agda-synthetic-categories#373 ·
Los mantenedores suelen responder en 1 día
-
agent-system-test-1319336938-43312257375-1- Agent System assignment-policy flow 43312257375-1Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
tanaabased/big-test-bucket#19952 ·
Los mantenedores suelen responder en 1 día