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

📁 feat: Move Archive Button to Conversation Menu (ChatGPT UI) #2743

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

Conversation

techwithanirudh
Copy link
Contributor

@techwithanirudh techwithanirudh commented May 16, 2024

Summary

Added the Archive button to the conversation menu for easier conversation management.

Note: ⚠️ This P.R. is one of multiple smaller P.R.s split from a larger one by GPT-4o.

Desktop

Before:
image

After:
image

Before:
image

Mobile

After:
image

Change Type

  • New feature (non-breaking change which adds functionality)

Testing

N/A

Test Configuration:

N/A

Checklist

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my own code
  • I have commented in any complex areas of my code
  • My changes do not introduce new warnings
  • Local unit tests pass with my changes

@techwithanirudh techwithanirudh marked this pull request as ready for review May 16, 2024 07:00
@techwithanirudh techwithanirudh changed the title Updating Archive Button feature with files from new/feat/new-ui 📁 feat: Move Archive Button to Conversation Menu May 16, 2024
@techwithanirudh techwithanirudh changed the title 📁 feat: Move Archive Button to Conversation Menu 📁 feat: Move Archive Button to Conversation Menu (ChatGPT UI) May 16, 2024
@JayDoubleu
Copy link
Sponsor

I actually like the fact that the archive icon is exposed directly on the chat title. In fact, I would vote to have the delete button there too!

@berry-13
Copy link
Collaborator

I actually like the fact that the archive icon is exposed directly on the chat title. In fact, I would vote to have the delete button there too!

well yes, but the title it's better to be more visibile imo

@JayDoubleu
Copy link
Sponsor

Tbh I very rarely come back to previous chats or read their titles , most of them are one offs which are deleted right after I run them to remove clutter. One could easily add menu resize ability that hides those buttons and adds them as you expand it for better readability

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.

None yet

3 participants