Vue - AutoScrollTop on pagination not working and pageChange event not emitted
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- frontend
Direzione di ricerca
Inizia da packages/vue/src/components/result/ReactiveList.jsx e analizza il watcher di hits e la relativa condizione di caricamento. Riproduci la paginazione con ReactiveList, quindi verifica che il cambio di pagina scorra verso l’alto per impostazione predefinita ed emetta l’evento pageChange.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- JavaScript
- Stelle
- 4.9k
- Fork
- 479
- Merge medio
- 4g 10h
- PR unite (30g)
- 4
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di appbaseio/reactivesearch
-
ci: add Node.js version matrix, caching, and lint step to test workflowForse già presa @Talha12Shiekh l’ha presa 70 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
appbaseio/reactivesearch#2322 ·
I maintainer di solito rispondono entro 1 giorno
-
chore: add Dependabot config for automated dependency updatesForse già presa @Talha12Shiekh l’ha presa 70 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
appbaseio/reactivesearch#2321 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
security: user-controlled URL params injected directly into Redux store via JSON.parseForse già presa @Talha12Shiekh l’ha presa 70 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
appbaseio/reactivesearch#2319 ·
I maintainer di solito rispondono entro 1 giorno
-
refactor: consolidate 12 near-identical SVG icon components into one patternForse già presa @Talha12Shiekh l’ha presa 72 giorni fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
appbaseio/reactivesearch#2324 ·
I maintainer di solito rispondono entro 1 giorno
-
refactor: SearchBox.js is 2692 lines — extract subcomponents for maintainabilityForse già presa @Talha12Shiekh l’ha presa 70 giorni fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
appbaseio/reactivesearch#2323 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di appbaseio/reactivesearch
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
Deepak3699/Ai_Mentor#244 ·
I maintainer di solito rispondono entro 1 giorno
-
ci-install-db-tools stall-case tests flake: stalled apt-get can be killed before it logs its callApertaeffort:low model:light plan planner:opus-5-5 tests
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
I maintainer di solito rispondono entro 1 giorno
-
Bug 🐞
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
mozilla-mobile/firefox-ios#35986 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
-
bug(sight): the dashboard's text truncations split surrogate pairs and show broken charactersApertacomponent:sight
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
agentic-os-org/ANOLISA#6738 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno