Skip to content

Improve AllChats delete confirmation UX#86

Open
Daksh54 wants to merge 1 commit into
avishek0769:mainfrom
Daksh54:feat/allchats-delete-modal
Open

Improve AllChats delete confirmation UX#86
Daksh54 wants to merge 1 commit into
avishek0769:mainfrom
Daksh54:feat/allchats-delete-modal

Conversation

@Daksh54
Copy link
Copy Markdown

@Daksh54 Daksh54 commented May 28, 2026

Summary

  • Replaced browser confirm() with an in-app delete confirmation modal
  • Added disabled/loading state with spinner and "Deleting..."
  • Removes deleted chats locally without full reload
  • Shows delete errors inside the modal

Testing

  • corepack pnpm exec eslint src/pages/AllChats.tsx
  • corepack pnpm build
  • Browser smoke-tested success, loading, and error states

@avishek0769
Copy link
Copy Markdown
Owner

@Daksh54 Can you please link the issue.

@Daksh54
Copy link
Copy Markdown
Author

Daksh54 commented May 29, 2026

#29

@avishek0769
Copy link
Copy Markdown
Owner

Looks good. Will merge it.

@avishek0769 avishek0769 added easy This issue is easy to solve SSoC26 Social Summer of Code - 2026 labels May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

easy This issue is easy to solve SSoC26 Social Summer of Code - 2026

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants