Skip to content

feat: ability to expand sidebar to see chats names #7816

Open
Abhijay007 wants to merge 4 commits intoblock:mainfrom
Abhijay007:feat/expandChat
Open

feat: ability to expand sidebar to see chats names #7816
Abhijay007 wants to merge 4 commits intoblock:mainfrom
Abhijay007:feat/expandChat

Conversation

@Abhijay007
Copy link
Collaborator

closes : #7791

Summary

Type of Change

  • Feature

AI Assistance

  • This PR was created or reviewed with AI assistance

Testing

Tested on desktop app

Screenshots/Demos (for UX changes)

expandChat

Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
@Abhijay007 Abhijay007 requested a review from alexhancock March 11, 2026 19:52
@Abhijay007 Abhijay007 changed the title Feat/expand chat feat: ability to expand sidebar to see chats names Mar 11, 2026
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 355d148484

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Copy link
Collaborator

@DOsinga DOsinga left a comment

Choose a reason for hiding this comment

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

can you use the settings.json to store this? we're trying to move away from localstorage

@Abhijay007
Copy link
Collaborator Author

P2 Badge Persist nav width after drag, not on every mousemove

Writing localStorage inside onMouseMove makes each pointer event perform a synchronous storage write, which blocks the UI thread and can make resize dragging noticeably janky on slower machines or busy render cycles. Keeping the width in React state during the drag and committing to storage in onMouseUp avoids this per-frame blocking while preserving persistence.

Useful? React with 👍 / 👎.

let me look into this

Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
@Abhijay007 Abhijay007 requested a review from DOsinga March 11, 2026 20:21
@Abhijay007
Copy link
Collaborator Author

@DOsinga updated it to use settings.json to store

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0ccc9cb554

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment on lines +253 to +255
<div className="w-full h-full overflow-hidden">
<Navigation />
</div>

Choose a reason for hiding this comment

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

P2 Badge Keep condensed nav overflow visible for hover controls

Wrapping Navigation in a new div with overflow-hidden clips UI that is intentionally rendered outside the nav bounds in condensed mode. In CondensedRenderer, the collapsed chat quick-action button is absolutely positioned with -right-9, so after this change that button is hidden and no longer clickable when hovering the chat row. This regresses an existing interaction in push/condensed vertical nav layouts.

Useful? React with 👍 / 👎.

Signed-off-by: Abhijay007 <Abhijay007j@gmail.com>
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.

Expand the left pane for longer chat names to remember the context

2 participants