Skip to content

frontend: compact/hidden chat threads #8323

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.

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

Merged
merged 11 commits into from
May 14, 2025
Merged

Conversation

haraldschilly
Copy link
Contributor

@haraldschilly haraldschilly commented May 5, 2025

  • this adds a button to the bottom of a thread to fold it. uses a new icon "to-top-outlined". To unfold, the icon is rotate by 180°.
  • this also unravels some of the chat message code, such that it is clearer which part belongs to what, moves a locally defined function in a function to be standalone, etc. far from perfect, but at least I can read the code much better.
  • I'm changing tooltip to use our "Tip" component, to make this more consistent.
  • I wanted to add a menu item to fold all LLM threads in one go, but I failed. I tried to model this after a jupyter notebook, but no luck. Instead, I added a button next to the video chat (with a smaller label for side chat).
  • Nothing changes behind the scenes, i.e. folding is based on an attribute of the root message.

screenshots

they show the threads as they are, partially closed, and well, the most interesting view is a long side chat with many LLM interactions, which can be closed with one button press.

Screenshot from 2025-05-13 17-00-05
Screenshot from 2025-05-13 16-59-39

side chat

Screenshot from 2025-05-13 16-59-07

side chat after clicking "fold LLM"

Screenshot from 2025-05-13 16-59-22

@haraldschilly haraldschilly force-pushed the chat-threads-smaller branch from 7f8e39e to c24dade Compare May 5, 2025 14:18
@haraldschilly haraldschilly force-pushed the chat-threads-smaller branch from ce34e40 to ca3feb9 Compare May 13, 2025 15:08
@haraldschilly haraldschilly marked this pull request as ready for review May 13, 2025 15:21
@williamstein
Copy link
Contributor

I wanted to add a menu item to fold all LLM threads in one go, but I failed. I tried to model this after a jupyter notebook, but no luck.

Jupyter has its own second level of complexity. The easiest way to do what you want would be to add a top-level action (in code-editor/actions.ts) called something like "foldAll". Make a command to do that that is a general generic menu item, like most of them, and then define foldAll in the actions for chat. it would in fact also be nice to have a fold all command for code editors using codemirror and code folding.

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

Successfully merging this pull request may close these issues.

2 participants