Skip to content
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

Tabs feature #4051

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Tabs feature #4051

wants to merge 6 commits into from

Conversation

ferenci84
Copy link
Contributor

@ferenci84 ferenci84 commented Feb 9, 2025

Description

Simple tab feature integrated with the current history. This is a minimal usable version of this feature. More features may be added.

Issue (opened by me):
#4030
(fork feature is not added)

Related issues:
#774
#1858

Checklist

  • The relevant docs, if any, have been updated or created
  • The relevant tests, if any, have been updated or created

Screenshots

Will provide image in a comment.

Testing instructions

The following user actions are handled:

  • User can add new tabs
  • User can close tabs, but it makes sure at least one tab is opened.
  • User can open existing history
  • Tab name is updated upon changing the session name
  • Adding new session in other way like Cmd+L or Cmd+Shit+L also adds new tab, clicking last session goes back to the previously opened tab.

Missing:

  • For a new tab, session name not updated before the user changes tabs
  • Model name is not assigned to the current tab - if you change model name on one tab, it will be the same on every other tabs too.

Implementation notes:

  • A single TabBar component is added, and a tabsSlice redux slice.
  • The existing Chat component is not changed, only new session, load session, save session is called to imitate tab changing.

Copy link

netlify bot commented Feb 9, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 8d2f390
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/67a8797feca0c900085654da

@ferenci84
Copy link
Contributor Author


Screenshot 2025-02-09 at 10 47 41

@ferenci84
Copy link
Contributor Author

Screenshot 2025-02-09 at 10 49 43

@mshostack
Copy link

Love it!

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