Repository navigation
feat(projects): add project detail page - #189
Merged
Merged
Conversation
Add /projects/[id] with the project header and one card per way to use it (web app, platform downloads, source code). Project cards now open this page. The backend only exposes the project link for now, so getProjectResources maps it to a web or source resource until per-platform downloads are available.
Contributor
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 9 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Use the Hero component with the projects page shapes, a gradient section title, linked CardIcon cards for each resource and a closing Glass panel to browse the other projects, with the shared section spacing.
Put the logo, category, title and description in a gradient Glass panel like the history page, show app icon logos without an extra frame, use the shared page top padding, max-w-330 column and section gaps, gradient section titles, and open external resource links in a new tab.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Aggiunge
/projects/[id]e collega le card della raccolta alla pagina del singolo progetto, con titolo, descrizione e logo dal backend, metadati dedicati, risorse disponibili e ritorno alla raccolta. Gli ID senza progetto mostrano la pagina non trovata.Il dettaglio usa una colonna di contenuti uniforme e i componenti/token già presenti: logo affiancato al testo su desktop, titolo da 36 a 57 px, spaziature più compatte, card delle risorse allineate e pulsanti a larghezza piena su mobile. Gestisce anche progetti senza logo o senza risorse e titoli lunghi. Non aggiunge piattaforme o link di download non forniti dai dati.
Autorizza il logo HTTPS dal repository
toto04/webeep-syncsu raw.githubusercontent.com e corregge l’altezza di tre sfondi looper della pagina Projects. Un test verifica il caricamento del logo con il loader reale Next.js e il rifiuto di origini non autorizzate.Verifica: TypeScript, Biome,
node --test tests/project-images.test.mjsegit diff --checkpassano. Controlli nel browser da 320 a 1440 px senza overflow, con logo caricato, pulsanti e collegamenti, titolo lungo, fallback senza logo e assenza di risorse. La verifica visiva del dettaglio usa un backend temporaneo con fixture perché il backend reale non è avviato su questa macchina; nessuna fixture è inclusa nel codice.