inbox icon indicating copy to clipboard operation
inbox copied to clipboard

feat: use react-virtuoso for convo message list

Open BlankParticle opened this issue 1 year ago • 2 comments

What does this PR do?

Replace @tanstack/react-virtual with react-virtuoso for convo messages view.

Type of change

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [ ] Chore (refactoring code, technical debt, workflow improvements)
  • [x] Enhancement (small improvements)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] This change requires a documentation update

Checklist

Required

  • [ ] Read Contributing Guide
  • [ ] Self-reviewed my own code
  • [ ] Tested my code in a local environment
  • [ ] Commented on my code in hard-to-understand areas
  • [ ] Checked for warnings, there are none
  • [ ] Removed all console.logs
  • [ ] Merged the latest changes from main onto my branch with git pull origin main
  • [ ] My changes don't cause any responsiveness issues

Appreciated

  • [ ] If a UI change was made: Added a screen recording or screenshots to this PR
  • [ ] Updated the UnInbox Docs if changes were necessary

BlankParticle avatar May 12 '24 15:05 BlankParticle

  • #448 Graphite
  • #447 Graphite
  • #446 Graphite 👈
  • #445 Graphite
  • #444 Graphite
  • main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @BlankParticle and the rest of your teammates on Graphite Graphite

BlankParticle avatar May 12 '24 15:05 BlankParticle

Graphite Automations

"Auto-assign PRs to author" took an action on this PR • (05/12/24)

1 assignee was added to this PR based on Rahul Mishra's automation.

graphite-app[bot] avatar May 12 '24 15:05 graphite-app[bot]

Merge activity

  • May 13, 2:17 PM EDT: @BlankParticle merged this pull request with Graphite.

BlankParticle avatar May 13 '24 18:05 BlankParticle