Skip to content

Fix Assets library chat toggle and sidebar image model chat controls#1602

Open
teleaziz wants to merge 1 commit into
mainfrom
fix-ssr-lib
Open

Fix Assets library chat toggle and sidebar image model chat controls#1602
teleaziz wants to merge 1 commit into
mainfrom
fix-ssr-lib

Conversation

@teleaziz

Copy link
Copy Markdown
Contributor

Summary

  • Fix Library route SSR/embedded detection so /library renders the correct chromeless picker state before hydration.
  • Keep the Library header and agent chat toggle available across normal Library pages.
  • Add the Assets image model picker to sidebar chat by passing imageModelMenu through AgentSidebar.
  • Share the Assets image model menu state between the Create page chat and sidebar chat.

Verification

  • Ran core typecheck.
  • Ran Assets typecheck.
  • Ran Assets production build, including SSR/Nitro build.
  • Verified with Playwright:
    • normal /library shows the chat toggle/header controls
    • embedded /library?embedded=1 renders chromeless picker UI
    • Create page chat and Library sidebar chat both show the Image model menu

@github-actions

Copy link
Copy Markdown
Contributor

Here's a visual recap of what changed:

Visual recap

Open the full interactive recap

@netlify

This comment has been minimized.

@netlify

This comment has been minimized.

@netlify

This comment has been minimized.

@netlify

This comment has been minimized.

@netlify

This comment has been minimized.

@netlify

This comment has been minimized.

@netlify

This comment has been minimized.

@netlify

This comment has been minimized.

@builder-io-integration

Copy link
Copy Markdown
Contributor

⚠️ Review Agent ran into a problem and couldn't finish reviewing the latest commit.

We've been automatically notified and are looking into it. Push a new commit to re-trigger the review, or contact support@builder.io if this keeps happening.

Error ID: 967412de6447473c9b91925a708d4141

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants