Vue - AutoScrollTop on pagination not working and pageChange event not emitted
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- frontend
Línea de trabajo
Comienza en packages/vue/src/components/result/ReactiveList.jsx e inspecciona el watcher de hits y su condición de carga. Reproduce la paginación con ReactiveList y luego verifica que cambiar de página desplaza el scroll hasta la parte superior de forma predeterminada y emite el evento pageChange.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Affected Projects
Vue.JS
Library Version:
1.29.0
Describe the bug
When clicking on a page number, the ReactiveList component should scroll to top (by default) and should send a pageChange event.
In my case, the scroll is not done and I never get the pageChange event.
I think I found the problem in the code of the /packages/vue/src/components/result/ReactiveList.jsx component:
hits(newVal, oldVal) {
this.$emit('data', this.getData());
if (this.shouldRenderPagination) {
// called when page is changed
if (this.isLoading && (oldVal || newVal)) {
if (this.hasPageChangeListener) {
this.$emit('pageChange', this.currentPageState + 1, this.totalPages);
this.$emit('page-change', this.currentPageState + 1, this.totalPages);
} else if (this.scrollOnChange) {
window.scrollTo(0, 0);
}
}
I can be wrong but I think that the line
if (this.isLoading && (oldVal || newVal)) {
should be
if (!this.isLoading && (oldVal || newVal)) {
It's missing a NOT before this.isLoading. Else it will never execute the code that is following the condition because the loading is finished since a while at this time.
- Lenguaje dominante
- JavaScript
- Estrellas
- 4.9k
- Forks
- 479
- Merge medio
- 4 d 10 h
- PR fusionados (30 d)
- 4
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 appbaseio/reactivesearch
-
ci: add Node.js version matrix, caching, and lint step to test workflowPosiblemente ocupada @Talha12Shiekh la tomó hace 71 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
appbaseio/reactivesearch#2322 ·
Los mantenedores suelen responder en 1 día
-
chore: add Dependabot config for automated dependency updatesPosiblemente ocupada @Talha12Shiekh la tomó hace 71 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
appbaseio/reactivesearch#2321 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
security: user-controlled URL params injected directly into Redux store via JSON.parsePosiblemente ocupada @Talha12Shiekh la tomó hace 71 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
appbaseio/reactivesearch#2319 ·
Los mantenedores suelen responder en 1 día
-
refactor: consolidate 12 near-identical SVG icon components into one patternPosiblemente ocupada @Talha12Shiekh la tomó hace 73 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
appbaseio/reactivesearch#2324 ·
Los mantenedores suelen responder en 1 día
-
refactor: SearchBox.js is 2692 lines — extract subcomponents for maintainabilityPosiblemente ocupada @Talha12Shiekh la tomó hace 71 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
appbaseio/reactivesearch#2323 ·
Los mantenedores suelen responder en 1 día
Todos los issues de appbaseio/reactivesearch
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
no-human-ai/no_human#659 ·
Los mantenedores suelen responder en 1 día
-
[BUG] Multi-day events show "Ended" while still in progressPosiblemente ocupada @tarunagnihotri534 la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
data-umbrella/du-event-board#231 · 2 comentarios ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
-
IO.get_env on Node truncates names at embedded NULPosiblemente ocupada @Yi-111-a la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
HigherOrderCO/Bend#1449 · 1 comentario ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
cryptpad/documentation#162 ·