Skip to content

Conversation

BrunoQuaresma
Copy link
Contributor

@BrunoQuaresma BrunoQuaresma commented May 19, 2025

What is done:

  • Redesign chat
  • Add support for light and dark themes
  • Use shadcn/ui

Before:

Screenshot 2025-05-19 at 17 52 43

Light:

Screenshot 2025-05-19 at 17 45 35

Dark:

Screenshot 2025-05-19 at 17 45 27

@BrunoQuaresma BrunoQuaresma requested a review from hugodutka May 19, 2025 20:47
@BrunoQuaresma BrunoQuaresma self-assigned this May 19, 2025
@hugodutka
Copy link
Collaborator

Love the new design! I spotted a couple of minor issues:

  1. The text returned by the api must use a mono font and exactly 80 columns to render correctly - that's the width of the underlying virtual terminal.
Screenshot 2025-05-20 at 10 18 10
  1. Key tiles overlap in control mode.
Screen.Recording.2025-05-20.at.10.19.46.mov
  1. Instead of displaying three dots for the user message while it's being sent maybe let's optimistically display the user message itself and move the three dots to the agent side?
Screen.Recording.2025-05-20.at.10.21.28.mov
  1. Could we pin the message box to the bottom of the page? I think that's how most AI chat interfaces do it.

  2. Loading dots for agent messages look a little off, like there's too little padding on them or maybe they are too small. You can see it for example with OpenAI Codex (OPENAI_API_KEY=xxx go run main.go server -- codex)

Screen.Recording.2025-05-20.at.10.36.27.mov

@BrunoQuaresma
Copy link
Contributor Author

  1. The text returned by the api must use a mono font and exactly 80 columns to render correctly - that's the width of the underlying virtual terminal.

It is using Geist Mono as mono font. What do you mean by "exactly 80 columns"? Does it mean, 80 letters?

  1. Key tiles overlap in control mode.

Good catch!

  1. Instead of displaying three dots for the user message while it's being sent maybe let's optimistically display the user message itself and move the three dots to the agent side?

Yes!

  1. Could we pin the message box to the bottom of the page? I think that's how most AI chat interfaces do it.

It should be pinned already. Let me fix that.

  1. Loading dots for agent messages look a little off, like there's too little padding on them or maybe they are too small. You can see it for example with OpenAI Codex (OPENAI_API_KEY=xxx go run main.go server -- codex)

Lets make it bigger!

Copy link
Collaborator

@hugodutka hugodutka left a comment

Choose a reason for hiding this comment

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

looks great! ship it

@BrunoQuaresma BrunoQuaresma merged commit ef3d380 into main May 21, 2025
1 check passed
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