Skip to content

scrollToMessage #27

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.

Sign up for GitHub

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

Jump to bottom
Merged
Parent: Beta
merged 60 commits into from
Jun 24, 2025
Merged

scrollToMessage #27

merged 60 commits into from
Jun 24, 2025

Conversation

@noaholsson
Copy link
Member

Description 📖

What?
A method for scrolling to a message by its ID has been added.

How?
This is done by making sure the message is fetched and then finding the index of the message and then scroll to the index.

This also exposes getItemLayout to optimise the scrolling.

Testing 🤞

  • I have tested the changes on iOS
  • I have tested the changes on Android

EnzoBomark and others added 30 commits July 25, 2023 14:23
@EnzoBomark
chore: release 0.4.4
369fcdc
@noaholsson
feat: added new prop setIsLoading
381aa25
@noaholsson
chore: release 0.5.0
286b520
@noaholsson
feat: performance improvements
9111d8d
@noaholsson
Merge pull request #16 from qteab/feat/performance-improvements
c4b36f5 
feat: performance improvements
@noaholsson
chore: release 0.5.1
b4e9f06
@noaholsson
fix: remove duplicate messages
27089b3
@noaholsson
Merge pull request #17 from qteab/fix/duplicate-messages
6e25066 
fix: remove duplicate messages
@noaholsson
chore: release 0.5.2
69b1d1c
@noaholsson
fix: remove duplicate messages
9a8460e
@noaholsson
Merge pull request #18 from qteab/fix/duplicate-messages
14a55cb 
fix: remove duplicate messages
@noaholsson
chore: release 0.5.3
93acf6e
@melvin-qte
fix: messages not being updated when last message is deleted
ab587c3
@melvin-qte
Merge pull request #19 from qteab/fix/last-message-deletion-bug
7ca2cf4 
fix: messages not being updated when last message is deleted
@melvin-qte
fix: bump used node version in ci
a492b94
@melvin-qte
fix: bump actions/cache@v2 -> actions/cache@v3
89e8841
@melvin-qte
fix: bump node version in nvmrc
41bdab4
@melvin-qte
Merge pull request #20 from qteab/fix/bump-ci-node-version
41610e6 
fix: bump used node version in ci
@melvin-qte
chore: release 0.5.4
5d74507
@melvin-qte
feat: rewrite caching system, use onScroll instead of onEndReached
d0c7a7f
@melvin-qte
fix: lint warnings
93ea088
@melvin-qte
fix: add clarifying comment to initial useEffect
48998b2
@melvin-qte
feat: do not fetch more if still loading previous messages
dd6f193
@melvin-qte
refactor: extract docToMessage to standalone function
31dbddf
@melvin-qte
refactor: extract prepareSnapShot to standalone function
d2939d3
@melvin-qte
refactor: extract appendSnapshot to standalone function
5bcd03a
@melvin-qte
refactor: move appendSnapshot to own util function
c391d5a
@melvin-qte
refactor: extract utils functions to own files
761fd56
@melvin-qte
refactor: move SnapshotChange type to prepareSnapshot.ts
bbe0dc9
@melvin-qte
docs: add better docs to util functions
a296390
melvin-qte and others added 28 commits April 30, 2025 13:18
@melvin-qte
fix: use get instead of onSnapshot in docToMessage
e5d51a3
@melvin-qte
refactor: docToMessage, remove non neccessary Promise instantiation
8107263
@melvin-qte
Merge pull request #21 from qteab/fix/better-msgs-caching
8344565 
feat: rewrite caching system, use onScroll instead of onEndReached
@melvin-qte
chore: release 0.5.5
8bf928a
@melvin-qte
feat: add way of determining if closeToTop
1269c00
@melvin-qte
feat: display scrollToBottomComponent if not close to top
e5414b8
@melvin-qte
feat: scroll to start of list on scrollToBottomComponent press
303497f
@melvin-qte
feat: add render support for newMessagesBannerComponent
2df5b6f
@melvin-qte
refactor: extract ChatFooter into own component and file
15a81ca
@melvin-qte
feat: separate all footer sections into own components
c23bc65
@melvin-qte
feat: only show newMessagesBanner if new messages have been registered
8e53189
@melvin-qte
fix: react and react native types mismatch
e5b4bf0
@melvin-qte
fix: build issues
5f8e1a1
@melvin-qte
fix: lint warnings
981322c
@melvin-qte
fix: arrow down does not mark as read
2802a7e
@melvin-qte
Merge pull request #23 from qteab/feat/new-messages-banner
c3438f2 
Feat/new messages banner
@melvin-qte
chore: release 0.5.6
45ae4c7
@noaholsson
feat: added passthrough for the maintainVisibleContentPosition
ebf0ce5
@noaholsson
Merge pull request #24 from qteab/feat/maintain-visible-content-position
2506750 
feat: added passthrough for the maintainVisibleContentPosition
@noaholsson
chore: release 0.5.7
9a82394
@noaholsson
fix: lower close to top threshold
e0b1f6f
@noaholsson
Merge pull request #25 from qteab/fix/lower-close-to-top-threshold
a3e5d8f 
fix: lower close to top threshold
@noaholsson
chore: release 0.5.8
ca849b7
@noaholsson
feat: updated react-native-firebase to v22
3a00de4 
BREAKING CHANGE: react-native-firebase ^22.0.0 is now required
@noaholsson
chore: correct deps
ce6c5eb
@noaholsson
Merge pull request #26 from qteab/feat/update-react-native-firebase
cdd19d8 
feat: updated react-native-firebase to v22
@noaholsson
chore: release 0.5.9
ba07edd
@noaholsson
feat: add scrollToMessage
0ac5c98
@noaholsson noaholsson changed the base branch from main to beta June 24, 2025 09:29
@noaholsson noaholsson merged commit 0599130 into beta Jun 24, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@noaholsson @EnzoBomark @melvin-qte