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 downwards pointing arrow #7

Open
odin- opened this issue Apr 23, 2017 · 6 comments
Open

Add downwards pointing arrow #7

odin- opened this issue Apr 23, 2017 · 6 comments

Comments

@odin-
Copy link

odin- commented Apr 23, 2017

Hangouts got a popup like bubble displaying "Jump to latest" at the bottom of the RecyclerView (scroll list). It got a down pointing arrow. So to be able to mimic that behaviour it would be nice if you could add a down pointing arrow.

(I can also do a pull req if you want me to).

@webianks
Copy link
Owner

You can position bubble according to your need. You can set it to bottom from your layout and also you can change the arrow according to your choice look for same in redme, rest of the behaviour remains the same as on click of that button recyclerview will be scrolled to top.

@webianks
Copy link
Owner

And I don't see any "Jump to latest" button in hangouts. Please provide screenshot.

@odin-
Copy link
Author

odin- commented Apr 24, 2017

Yup I've added my own down pointing arrow. But to mimic Hangouts out of the box it might be nice if you included it default in your library.
c3izajzwmaqptru

@webianks
Copy link
Owner

Okay cool I got that. I will add this behaviour.

@odin-
Copy link
Author

odin- commented Apr 24, 2017

In that case I guess this code in onTouchEvent is problematic since it auto scrolls to position 0. I would like to scroll to last position:

                if (recyclerView != null)
                     recyclerView.smoothScrollToPosition(0);

@webianks
Copy link
Owner

I will need to this also to check the behaviour of recyclerview. I am sure that in the chat section you have reverse layout meaning last element becomes first element and then the above code should not raise any issue but it needs to be tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants