Skip to content

fix:change comptuer use page url to openai #13

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix:change comptuer use page url to openai #13

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions app/page.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ export default function Home() {
<nav className="flex justify-between items-center px-8 py-4 bg-white border-b border-[#CAC8C7] z-10">
<div className="flex items-center gap-3">
<a
href="https://www.browserbase.com/computer-use"
href="https://www.browserbase.com/cua/openai"
Copy link

Choose a reason for hiding this comment

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

The URL update in the main page creates inconsistency with similar links in the ChatFeed component that still use the old URL structure.

View Details
📝 Patch Details 
diff --git a/app/components/ChatFeed.tsx b/app/components/ChatFeed.tsx
index 69f4297..25990e0 100644
--- a/app/components/ChatFeed.tsx
+++ b/app/components/ChatFeed.tsx
@@ -1306,7 +1306,7 @@ export default function LegacyChatFeed({
       >
         <div className="flex items-center gap-2">
           <a
-            href="https://browserbase.com/computer-use"
+            href="https://www.browserbase.com/cua/openai"
             target="_blank"
             rel="noopener noreferrer"
             className="flex items-center gap-3 hover:opacity-90 transition-opacity duration-200"
@@ -1325,7 +1325,7 @@ export default function LegacyChatFeed({
         </div>
         <div className="flex items-center gap-2">
           <a
-            href="https://browserbase.com/computer-use"
+            href="https://www.browserbase.com/cua/openai"
             target="_blank"
             rel="noopener noreferrer"
           >

Analysis

Inconsistent branding URLs in navigation components

What fails: Navigation elements (logo links and Deploy buttons) in app/page.tsx and app/components/ChatFeed.tsx pointed to different Browserbase URLs, creating inconsistent user experience

How to reproduce:

  1. Load the main page - observe logo and Deploy button point to https://www.browserbase.com/cua/openai
  2. Start chat session to open ChatFeed component - observe identical logo and Deploy button point to https://browserbase.com/computer-use
  3. Users clicking the same visual elements get different destinations

Result: Same branding elements (CUA Browser logo and Deploy button) navigate to different URLs depending on component

Expected: Identical navigation elements should consistently point to the same destination per UI consistency best practices

Fixed: Updated ChatFeed.tsx URLs to match page.tsx URLs (/cua/openai) for consistent navigation experience

target="_blank"
rel="noopener noreferrer"
className="flex items-center gap-3 hover:opacity-90 transition-opacity duration-200"
Expand All @@ -139,7 +139,7 @@ export default function Home() {
</div>
<div className="flex items-center gap-2">
<a
href="https://www.browserbase.com/computer-use"
href="https://www.browserbase.com/cua/openai"
target="_blank"
rel="noopener noreferrer"
>
Expand Down