The last example on "How to Create a Basic Plugin" returns a different result than the previous example.
@Jeff-Schafer ya está trabajando en esto.
Desde el 16/1/2017.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 55/100
- Tipo de issue
- Documentación
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, jquery
- Área
- documentation
Línea de trabajo
Abre la lección «Basic Plugin Creation» en la URL enlazada y compara el ejemplo optimizado final con los ejemplos anterior y primero de «Putting It Together». Actualiza el ejemplo y la nota sobre cómo recuperar href para que el resultado mostrado y la explicación coincidan, y luego verifica que los ejemplos de la lección produzcan la salida documentada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The last example on https://learn.jquery.com/plugins/basic-plugin-creation/ says:
"Our plugin can be optimized though:"
and shows the "optimized" code.
But the example before it, when run, just appends the text in the href for that tag, & this "optimized" code appends the full path to where the .html file is to what is in the href for that tag.
The first example under the "Putting It Together" correctly appends "page.html" to the text between the tags.
---> Foo (page.html)
Using a local .html file (like on the Desktop), the "optimized" returns:
---> Foo file:///C:/users/..../Desktop/page.html (without the parentheses, unlike in the first example).
Below the last example, it says:
"Notice also that we're not using the .attr() method to retrieve the href attribute, because the native DOM API gives us easy access with the aptly named href property."
To make it have the same results as the first example in this section "this.href" needs to be replaced with "$(this).attr("href")".
This will contradict the text mentioned above ("not using the .attr() method"), and needs to be updated.
- Lenguaje dominante
- JavaScript
- Estrellas
- 914
- Forks
- 472
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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/learn.jquery.com
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
jquery/learn.jquery.com#828 ·
-
Word(s) missing in article?Quizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. AbiertoChapter: events
Dificultad 1/5 Menos de una hora Aptitud para principiantes 70/100
jquery/learn.jquery.com#751 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
jquery/learn.jquery.com#814 · 1 comentario ·
-
Chapter: jquery-core help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
jquery/learn.jquery.com#799 ·
-
Chapter: plugins
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
jquery/learn.jquery.com#796 ·
Todos los issues de jquery/learn.jquery.com
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
good first issue help wanted javascript up-for-grabs
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
andrewsiah/agentswarm-data-tasks#307 ·
Los mantenedores suelen responder en 1 día
-
needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
coder/claudecode.nvim#588 ·
Los mantenedores suelen responder en 1 día
-
area: web tech debt
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
digitalbazaar/http-client#60 ·