refactor(site): modernize create organization page - #27914
Conversation
Sorry, something went wrong.
Documentation CheckUpdates Needed
Automated review via Coder Agents |
Sorry, something went wrong.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: aa1084fbe6
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Sorry, something went wrong.
| <Link to="/organizations"> | ||
| <Button variant="outline" type="button"> | ||
| Cancel |
There was a problem hiding this comment.
Render the cancel link as the button root
When keyboard or screen-reader users reach the form actions, this renders a focusable <button> inside a focusable <a>, producing invalid nested interactive markup and two tab stops for the same action. Use the same Button asChild pattern as the back link so the router link itself receives the button styling.
AGENTS.md reference: site/AGENTS.md:L18-L19
Useful? React with 👍 / 👎.
Sorry, something went wrong.
|
@jakehwll could you include the Figma link in the description? |
Sorry, something went wrong.
There is no Figma, im just aligning to the rest of the "Create" pages 🙂 |
Sorry, something went wrong.
|
@jakehwll regarding the back button placement, I think the create workspace page is a better pattern to follow. When there is no left sidebar, I think having the back button use that space is a bit more consistent. |
Sorry, something went wrong.
|
@jakehwll I agree with with Jaayden's susggestion to have the back button follow the Create Workspace pattern that has the back outside the content column. Any chance we could get that change in this PR? 
|
Sorry, something went wrong.
Resolves conflicts in the create organization page after main modernized the premium paywall (canViewPremium) and removed IconPickerField in favor of the shared IconField.
|
Fair enough, I was battling the overflow the old page had on mobile. This would have meant the Back to organizations button would've overflowed into the content.
|
Sorry, something went wrong.
Reviewers
At least 1 approving review is required to merge this pull request.
Modernizes the create organization page to match the AI Providers / OAuth2 create-page pattern.
The page is now a thin permission shell; the view owns the create mutation, uses SettingsHeader with a live icon preview, IconPickerField, field descriptions, and a bordered form card in a centered container. Storybook play coverage and e2e selectors are updated for the new submit label. Unentitled deployments still hit PaywallPremium.