Skip to content

Restyle deep research dashboard#94

Open
shriyatheunicorn wants to merge 2 commits into
add-deep-research-agent-templatefrom
deep-research-agent-dashboard-ui
Open

Restyle deep research dashboard#94
shriyatheunicorn wants to merge 2 commits into
add-deep-research-agent-templatefrom
deep-research-agent-dashboard-ui

Conversation

@shriyatheunicorn
Copy link
Copy Markdown

@shriyatheunicorn shriyatheunicorn commented May 12, 2026

Splits the GTM demo-inspired dashboard restyle and Browserbase logo update out of the base deep research agent template PR.\n\nChanges:\n- Restyles the deep research dashboard after the Browserbase GTM research demo visual language.\n- Adds the official Browserbase logo asset.\n- Keeps the existing API behavior and local/Vercel fallback logic unchanged.\n\nVerification:\n- npm run check\n- inline script parse checks\n- local in-app browser smoke test


Note

Low Risk
Low risk UI-only change: updates static HTML/CSS and adds a logo asset without touching API calls or research logic.

Overview
Restyles the deep-research-agent static dashboard (public/index.html) to match a Browserbase GTM demo-inspired visual theme, including updated color tokens, layout tweaks, and component styling.

Adds the official Browserbase logo SVG (public/browserbase-logo.svg), renders it in the sidebar header, and introduces a small “research stages” strip plus minor copy adjustments in the explainer section.

Reviewed by Cursor Bugbot for commit fba2b10. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit fba2b10. Configure here.


.side-nav {
grid-template-columns: repeat(4, minmax(0, 1fr));
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CSS cascade causes side-nav mobile layout regression

Medium Severity

The new @media (max-width: 980px) rule for .side-nav at line 1232 appears later in the stylesheet than the original @media (max-width: 680px) rule at line 807 that sets .side-nav to grid-template-columns: 1fr. At viewports below 680px both media queries are active, and the later 980px rule wins the cascade, keeping .side-nav locked at 4 columns instead of collapsing to 1. The new 680px media query omits .side-nav, so there's nothing to counteract this.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit fba2b10. Configure here.

<div class="app-shell">
<aside class="sidebar">
<div class="brand">
<img class="brand-logo" src="browserbase-logo.svg" alt="" />
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logo SVG unreachable on local dev server

Low Severity

The new <img src="browserbase-logo.svg"> resolves to /browserbase-logo.svg, but handleDashboardRequest in api-utils.ts only serves /, /index.html, /health, and /research—returning a 404 JSON response for all other paths. The logo won't load when using the local dev server (npm run web). On Vercel the public/ directory is served at root so it works there, but local development shows a missing image.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit fba2b10. Configure here.

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.

1 participant