Gallery search and filters stop working after you come back to /templates through a site link
I maintainer di solito rispondono entro 1 giorno
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
The gallery's search box and category pills stop working once you come back to /templates through a link on the site. On https://og.codercops.com:
- Open
/templates. Typing "dark" filters the gallery, and the GitHub pill shows 10 templates. - Click "Preview" in the header, then "Templates".
- Type "dark" or click a category pill. All 110 templates stay on screen, the "Showing" line doesn't change, and there's no error in the console.
The site uses Astro's <ClientRouter /> (src/layouts/Layout.astro, line 75), so links inside the site swap the page without a full reload. The router runs each script once until the next full reload: it remembers scripts by their text and skips any it has already run, unless the script has data-astro-rerun. The gallery's code is a top-level <script is:inline> (src/pages/templates.astro, lines 80 to 163), so the second time the page is shown, nothing attaches listeners to the new search box and pills, and ?q= and ?category= aren't read either.
What to do
- Put the gallery setup in a function and run it on
astro:page-load, which fires on the first load and after every navigation.Layout.astrodoes this for the scroll animations (line 107). - The event fires on every page, so return early when the gallery isn't on the page.
- Attach the listeners once per visit to
/templates, not once more for every navigation. - If you'd rather add
data-astro-rerun, wrap the code in a function first: run twice, the script's top-levelconsts would be declared twice and it would throw. - Keep the search matching names, descriptions and tags (#147), and keep
?q=and?category=working.
Where to look
src/pages/templates.astro: the script at lines 80 to 163src/layouts/Layout.astro:<ClientRouter />(line 75) and theastro:page-loadlistener (line 107)src/components/Header.astro(line 11) andsrc/components/Footer.astro(line 20) link to/templates
Acceptance criteria
- After going from
/templatesto another page and back with the header or footer link, the search box and the category pills filter the gallery, and the "Showing" line updates - Going back and forth several times still works, without adding another set of listeners each time
- Opening
/templates?q=dark&category=githubdirectly still applies both -
npm run check,npm run testandnpm run buildpass - The PR says how you tested the navigation in a browser with
npm run dev
Want to work on this? Comment below and I'll assign it to you. Please take one issue at a time, branch off dev, and open your PR against dev with Closes #<this issue number>.
- Lingua principale
- TypeScript
- Stelle
- 8
- Fork
- 4
- Merge medio
- 6h 9m
- PR unite (30g)
- 66
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 codercops/ogcops
-
accessibility bug good first issue hacktoberfest ui
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
accessibility bug good first issue hacktoberfest
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
I maintainer di solito rispondono entro 1 giorno
-
bug good first issue hacktoberfest ui
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
bug good first issue hacktoberfest
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
codercops/ogcops#167 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
An empty `imageUrl=`, `logoUrl=` or `template=` returns an error instead of the defaultForse già presa @Saifjafri212213 l’ha presa 1 giorno fa. Apertabug good first issue hacktoberfest
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
codercops/ogcops#166 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di codercops/ogcops
Issue simili
-
native
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
prime-radiant-inc/evener#4839 ·
I maintainer di solito rispondono entro 1 giorno
-
bug needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
processone/fluux-messenger#1790 ·
I maintainer di solito rispondono entro 1 giorno
-
🛣️ Infrastructure
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
opencrvs/opencrvs-core#14232 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
kunchenguid/gh-axi#220 ·
I maintainer di solito rispondono entro 1 giorno