feat: add Coder registry links to template creation and editing #18680
+35
−5
Conversation
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
Add links to https://registry.coder.com in two locations: - "Browse Templates" card on starter templates page - "Browse Modules" button in template editor topbar This provides users with easy access to community templates and modules as a short-term solution before full registry integration. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Replace template card with "Browse the Coder Registry" link in page header. The link appears in the top right aligned with the page title and includes an external link icon to indicate it opens in a new tab. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Replace "Browse Modules" button with "Browse the Coder Registry" link in the template editor topbar. The link matches the styling of the starter templates page with an external link icon. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Replace custom styled links with the existing Link component that: - Provides consistent styling and behavior - Includes proper external link icon (SquareArrowOutUpRightIcon) - Has built-in accessibility features and focus states - Removes duplicate custom CSS for link styling 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Add simple tests to verify the Coder registry links are present and properly configured on both the starter templates page and template editor: - Tests link text, href, target, and rel attributes - Uses existing test patterns with MSW and React Testing Library - Verifies external link behavior and accessibility 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Fix TypeScript error by passing createTestQueryClient() to renderEditorPage function call in the registry link test, matching the pattern used by other tests in the file. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
f0ssel
force-pushed
the
f0ssel/registry-branch
branch
from
5060fa3 to
f72fbd5
Compare
- Move template editor registry link to left of build status pill - Make links consistent with size=\sm\ variant - Add white-space: nowrap to prevent text wrapping - Add proper spacing with margins
aslilac
reviewed
Jun 30, 2025
There was a problem hiding this comment.
- suggested some tailwind classes to use instead of emotion styles
- I feel like the tests are pretty low value. they aren't verifying any actual behavior, they're just repeating stuff that the code already says explicitly. chromatic snapshots show that the link is definitely there, and will catch if it ever changes for "free" (no testing code).
site/src/pages/CreateTemplateGalleryPage/CreateTemplateGalleryPageView.tsx
Outdated
Show resolved
Hide resolved
site/src/pages/CreateTemplateGalleryPage/CreateTemplateGalleryPageView.tsx
Outdated
Show resolved
Hide resolved
site/src/pages/TemplateVersionEditorPage/TemplateVersionEditor.tsx
Outdated
Show resolved
Hide resolved
matifali
reviewed
Jul 1, 2025
…n links - Replace custom CSS with Tailwind classes (whitespace-nowrap, mr-2) - Add 'button' prop to Link components for button-like styling - Apply consistent styling across template editor and template gallery pages
- Replace Link components with Button + asChild pattern - Use variant=\outline\ for button styling - Keep tailwind classes for proper spacing
- Remove registry link tests from both page components - Fix Button variant and size values to use valid options - Remove unused theme import and reference - Fix build issues
- Import ExternalLinkIcon from lucide-react - Add icon after text in both registry link buttons - Use flex layout for proper alignment in the buttons
|
Updated the styling and code from feedback 
|
matifali
approved these changes
Jul 1, 2025
|
/cherry-pick release/2.24 |
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.
Summary
🤖 Generated with Claude Code