feat(frontend): templates pages join the shared layout - #5846
Conversation
dosubot
Bot
added
the
size:L
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
af1f12a to
f1342a5
Compare
dosubot
Bot
added
size:XL
dosubot
Bot
added
size:L
Railway Preview Environment
Updated at 2026-08-09T21:22:44.963Z |
- Routing: /apps/agent-templates parsed 'agent-templates' as an app id, dropping the gallery into the first app's context (the 'opens the first agent' bug). It joins the reserved app-collection routes. - Gallery: standard page shape - title at top in the shared column with a description line (PageLayout grows an opt-in description prop), then two columns on one background, no tinted rail, no divider: category filters left, search + grid right. - Detail: top-and-bottom - name and back in a top bar, body below, no side-by-side layout.
/agent-templates* is a full-height route, so content below the fold was unreachable at narrow widths. The body scrolls; the back link and the use action stay pinned.
b9f648f to
1518387
Compare
f1342a5 to
fd1cb5e
Compare
1 participant
Mahmoud's templates rework, three parts:
The "opens the first agent" bug
/apps/agent-templatessat where an app id sits in the route, so the app-state parser treatedagent-templatesAS an app id — putting the whole gallery in an app's context (agent sidebar, bogususeAppId(), and the evaluator templates leaking into view). The path joinsRESERVED_APP_COLLECTION_ROUTESbesidearchived, so the gallery is a proper project-level page now.Gallery layout
Standard page shape: title at the top in the shared centered column with the page description under it (
PageLayoutgrows an opt-indescriptionprop — pages without one are pixel-unchanged), then two columns on ONE background — no tinted rail, no divider line. Category filters left; search + template grid right.Detail layout
Top-and-bottom instead of side-by-side: a top bar with the template's name and back to the gallery; body below.
Layout only — template functionality untouched.