Gallery search and filters stop working after you come back to /templates through a site link
Les mainteneurs répondent en général sous 1 jour
Évaluation
Cette issue n'a pas encore été évaluée.
Description
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>.
- Langage dominant
- TypeScript
- Étoiles
- 8
- Forks
- 4
- Merge moyen
- 6 h 9 min
- PR mergées (30 j)
- 66
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de codercops/ogcops
-
accessibility bug good first issue hacktoberfest ui
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
accessibility bug good first issue hacktoberfest
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue hacktoberfest ui
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue hacktoberfest
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
codercops/ogcops#167 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
An empty `imageUrl=`, `logoUrl=` or `template=` returns an error instead of the defaultPeut-être pris @Saifjafri212213 l’a pris aujourd’hui. Ouvertebug good first issue hacktoberfest
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
codercops/ogcops#166 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de codercops/ogcops
Issues similaires
-
agent/scanner help wanted hive/claimed-by-scanner kind/bug stage/020-claimed
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
hivecommons/rationguard#315 ·
Les mainteneurs répondent en général sous 1 jour
-
agent-ready area: ui-sdk priority: p3 type: chore
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
schlessera/brain-kit#2534 ·
Les mainteneurs répondent en général sous 1 jour
-
bug lane:gh-5
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
LLM-4-People/bridlewise-testing#122 ·
Les mainteneurs répondent en général sous 1 jour
-
bug priority: P2
Difficulté 1/5 Moins d'une heure Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
enhancement priority:low ready-for-dev
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
Les mainteneurs répondent en général sous 1 jour