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

Add menu point to scroll chat to bottom #324

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rebootl
Copy link

@rebootl rebootl commented Jun 8, 2022

Since i sometimes get a lot of messages from my IRC bouncer, the loading takes some time and then i need an easy way to scroll down.

Probably only useful in my special use case. Leaving it here since it may be useful for someone.

Also not sure about the code cause i have no experience on Android.

@MCMrARM
Copy link
Owner

MCMrARM commented Feb 17, 2023

Hello and sorry for the delayed reploy,

If you make this a setting (off by default), I'll merge this.

The way I did this in message view v2 was to add a small down arrow in bottom right corner that shows if you scroll up (similar to many other apps). You can find the code for this here:
https://github.com/MCMrARM/revolution-irc/blob/newui/app/src/main/res/layout/chat_messages_fragment.xml#L56-L62
https://github.com/MCMrARM/revolution-irc/blob/newui/app/src/main/java/io/mrarm/irc/view/JumpToRecentButton.java
If you like this idea, you can convert your code to do this instead, otherwise an option will suffice.

@celeron55
Copy link

Uh, how is using a bouncer on a large channel a special use case? I'm having this problem also. It's a weird problem to have on an otherwise well polished application.

@rebootl
Copy link
Author

rebootl commented Sep 19, 2024

Hi,
thanks for your feedback :)
Sorry this has been open for a long time, I may be able to find some time and make the changes so that this can merged, it may take some time though...

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.

3 participants