Skip to content

Instantly share code, notes, and snippets.

@chrisbanes
Last active January 6, 2024 10:10
Show Gist options
  • Save chrisbanes/8391b5adb9ee42180893300850ed02f2 to your computer and use it in GitHub Desktop.
Save chrisbanes/8391b5adb9ee42180893300850ed02f2 to your computer and use it in GitHub Desktop.
Fixed AppBarLayout.Behavior for https://issuetracker.google.com/66996774
/*
* Copyright (C) 2017 The Android Open Source Project
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package your.package;
import android.content.Context;
import android.support.design.widget.AppBarLayout;
import android.support.design.widget.CoordinatorLayout;
import android.support.v4.view.ViewCompat;
import android.util.AttributeSet;
import android.view.View;
/**
* Workaround AppBarLayout.Behavior for https://issuetracker.google.com/66996774
*
* See https://gist.github.com/chrisbanes/8391b5adb9ee42180893300850ed02f2 for
* example usage.
*
* Change the package name as you wish.
*/
public class FixAppBarLayoutBehavior extends AppBarLayout.Behavior {
public FixAppBarLayoutBehavior() {
super();
}
public FixAppBarLayoutBehavior(Context context, AttributeSet attrs) {
super(context, attrs);
}
@Override
public void onNestedScroll(CoordinatorLayout coordinatorLayout, AppBarLayout child, View target,
int dxConsumed, int dyConsumed, int dxUnconsumed, int dyUnconsumed, int type) {
super.onNestedScroll(coordinatorLayout, child, target, dxConsumed, dyConsumed,
dxUnconsumed, dyUnconsumed, type);
stopNestedScrollIfNeeded(dyUnconsumed, child, target, type);
}
@Override
public void onNestedPreScroll(CoordinatorLayout coordinatorLayout, AppBarLayout child,
View target, int dx, int dy, int[] consumed, int type) {
super.onNestedPreScroll(coordinatorLayout, child, target, dx, dy, consumed, type);
stopNestedScrollIfNeeded(dy, child, target, type);
}
private void stopNestedScrollIfNeeded(int dy, AppBarLayout child, View target, int type) {
if (type == ViewCompat.TYPE_NON_TOUCH) {
final int currOffset = getTopAndBottomOffset();
if ((dy < 0 && currOffset == 0)
|| (dy > 0 && currOffset == -child.getTotalScrollRange())) {
ViewCompat.stopNestedScroll(target, ViewCompat.TYPE_NON_TOUCH);
}
}
}
}
AppBarLayout abl = findViewById(R.id.app_bar);
((CoordinatorLayout.LayoutParams) abl.getLayoutParams()).setBehavior(new FixAppBarLayoutBehavior());
<android.support.design.widget.CoordinatorLayout xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto"
android:layout_width="match_parent"
android:layout_height="match_parent">
<android.support.design.widget.AppBarLayout
android:id="@+id/app_bar"
android:layout_height="..."
android:layout_width="..."
app:layout_behavior="your.package.FixAppBarLayoutBehavior">
</android.support.design.widget.AppBarLayout>
<!-- Content -->
</android.support.design.widget.CoordinatorLayout>
@sjd753
Copy link

sjd753 commented May 24, 2018

It solves the fling problem but normal ACTION_MOVE does not work on RecyclerView (Horizontal) inside NestedScrollView

@chongbo2013
Copy link

thank u

@booncol
Copy link

booncol commented Jun 17, 2018

You are genius! Thank you.

@rawewhat
Copy link

It works perfectly, but when I test on older Android like 4.1.1 the problem happens again, the button can't be click and only can scroll half of the screen.

@ljaniszewski
Copy link

yeah, you can't expect this to be working on such old Android.
For me works good, thanks!

@luangs7
Copy link

luangs7 commented Jun 29, 2018

Works perfect to me! Thanks, you saved me

@dariush-fathie
Copy link

thank you . it works

@nobeginning
Copy link

Nice, thank you man

@Ahmedfir
Copy link

Amazing ! I lost a lot of time to recognize that it was the appbar... thanks a lot !

@rodrigosimoesrosa
Copy link

Works perfectly to me, thank you very much!

@lvaizer
Copy link

lvaizer commented Aug 9, 2018

Thanks!!!

@jhonsonkilly
Copy link

Thanks

@ibrahimsn98
Copy link

You saved me :D Thanks!

@longjiang-dragon
Copy link

Thanks!!!

@hellaandrew
Copy link

Excellent work! This saved me a lot of headache.

@dwist14
Copy link

dwist14 commented Sep 13, 2018

Mantap jiwa..

@wangkunlin
Copy link

Works perfect! Thanks

@ShahoodulHassan
Copy link

Not working for me on V27.1.1
I've tried both java and xml ways but in vain.
Checked on emulator running on 5.0 and physical device running on 7.1.1
Any ideas ?

@fazalurrehmansharif
Copy link

Thanks so much i was wondering if there is any solution or not but now i am feeling very lucky thankyou so much

@shubhamvashisht
Copy link

Am having same issue with BottomSheetBehavior, any fix for that?

@harsha-main
Copy link

Why isn't this incorporated in the official release so far? I have a Horizontal recyclerview inside a vertical recyclerview, which is inside a nestedscrollview. I'm now having issues with the usability of the horizontal scroll. It works very different from the Google Play app, which has good UX, I need to know if there's a standard solution for this provided by Android.

@geoff-powell
Copy link

geoff-powell commented Jun 21, 2019

@harsha-main I have the same layout with horizontal recycler inside a vertical recycler inside a coordinator layout. One think that fixed my issues with the app bar layout was setting android:nestedScrollEnabled="false" on the horizontal recycler. It fixed our issues with our collapsing header so it might fix your issues.

@allenjy
Copy link

allenjy commented Oct 23, 2019

Thanks!!!

@Ekt0s
Copy link

Ekt0s commented Nov 19, 2019

Thanks Chris ! 🙌

@fjz97
Copy link

fjz97 commented Jan 10, 2020

Thanks!

@jesselima
Copy link

Oh God!!!
Finaly!!!
Thanks!!!!

@sajjadyousefnia
Copy link

sajjadyousefnia commented May 26, 2020

doesn't work for me, crashes!.

@boylenssen
Copy link

Seems to be solved in the latest version of AndroidX dependencies

@yummyChina
Copy link

doesn't work for me

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