In the current version it is possible to change the animation for the dialog as whole, however, it not possible to change the layout animation for the BottomSheet.List. It would be nice if it was a styleable attr.

When I swipe down with this menu I get the following error:

Attempting to destroy the window while drawing!

When I try to click on my Button to show the dialog again it doesn't show, the screen simply flashes black for a split second. When I click on the Button again it shows properly. The only way I was able to get this to work is by doing this for canceling:

closeOptionsMenu() and sheet.setCanceledOnSwipeDown(false);

Is there a way to get the swipe down working properly?

For more flexibility this would allow to use this library in more or less any case. For example in my app I need to be able to show a variable list of items depending on the current state of the app.

• Allow usage of RecyclerView (we get better Lists and Grids!)
• Allow Custom Adapters
• Allow setting own View
• Adjust callbacks for above (maybe multiple callbacks or custom callbacks?)

I will most likely start working on this soon :). If you have any things you want to tell me to do while doing it, or what to watch out for, please let me know. Of course I will send a pull request in the end.

This might go into the same pull request as #24 because it will probably as well change the Dialog to a DialogFragment.

The latest support library version (23.2.0) adds Google's own variety of BottomSheets and unfortunately there is name clash on the style attribute when you have both libraries as dependencies meaning the app fails to compile.

Currently when in Landscape mode the sheet quickly fills up the entire screen. Even worse, some action items are cut off. I could go to Grid mode, but on small screens even that won't help.

Hi,

could you add an option to set the maxWidth of the BottomSheet. Right now, on landscape mode it fill up the whole screen. I would like it to be more narrow (just like the menu options)

thanks

Hey, i'm using the latest Android O version on Pixel

happens everytime i try to open it

E/AndroidRuntime: FATAL EXCEPTION: main Process: com.tnx.apps.coinportfolio, PID: 2885 java.lang.AbstractMethodError: abstract method "android.view.MenuItem android.support.v4.internal.view.SupportMenuItem.setAlphabeticShortcut(char, int)" at android.view.MenuInflater$MenuState.setItem(MenuInflater.java:477) at android.view.MenuInflater$MenuState.addItem(MenuInflater.java:541) at android.view.MenuInflater.parseMenu(MenuInflater.java:193) at android.view.MenuInflater.inflate(MenuInflater.java:114) at com.cocosw.bottomsheet.BottomSheet$Builder.sheet(BottomSheet.java:531) at com.tnx.apps.coinportfolio.activity.ManageTransactionsActivity$MyAdapter$1.onClick(ManageTransactionsActivity.java:126) at android.view.View.performClick(View.java:6256) at android.view.View$PerformClick.run(View.java:24697) at android.os.Handler.handleCallback(Handler.java:789) at android.os.Handler.dispatchMessage(Handler.java:98) at android.os.Looper.loop(Looper.java:164) at android.app.ActivityThread.main(ActivityThread.java:6541) at java.lang.reflect.Method.invoke(Native Method) at com.android.internal.os.Zygote$MethodAndArgsCaller.run(Zygote.java:240) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:767)

I would suggest the following well known and widely used solution instead of the ClosableSlidingLayoutyou now have (actually, the horizontal swipe could be removed from it completely). This handles the swipe easily and nicely and could be used for both directions, to open up from More and to close down:

public class SwipeFrameLayout extends android.widget.FrameLayout {
  private final GestureDetector gestureDetector;
  protected OnHorizontalSwipe horizontalListener;
  protected OnVerticalSwipe verticalListener;

  public interface OnHorizontalSwipe {
    public boolean onSwipeLeft();

    public boolean onSwipeRight();
  }

  public interface OnVerticalSwipe {
    public boolean onSwipeUp();

    public boolean onSwipeDown();
  }

  public SwipeFrameLayout(Context context) {
    super(context);
    gestureDetector = new GestureDetector(context, new GestureListener());
  }

  public SwipeFrameLayout(Context context, AttributeSet attrs) {
    super(context, attrs);
    gestureDetector = new GestureDetector(context, new GestureListener());
  }

  public SwipeFrameLayout(Context context, AttributeSet attrs, int defStyleAttr) {
    super(context, attrs, defStyleAttr);
    gestureDetector = new GestureDetector(context, new GestureListener());
  }

  @Override
  public boolean dispatchTouchEvent(MotionEvent e) {
    gestureDetector.onTouchEvent(e);
    return super.dispatchTouchEvent(e);
  }

  public void setHorizontalListener(OnHorizontalSwipe horizontalListener) {
    this.horizontalListener = horizontalListener;
  }

  public void setVerticalListener(OnVerticalSwipe verticalListener) {
    this.verticalListener = verticalListener;
  }

  private final class GestureListener extends SimpleOnGestureListener {
    private static final int SWIPE_DISTANCE_THRESHOLD = 100;
    private static final int SWIPE_VELOCITY_THRESHOLD = 100;

    @Override
    public boolean onFling(MotionEvent e1, MotionEvent e2, float velocityX, float velocityY) {
      float dX = e2.getX() - e1.getX();
      float dY = e2.getY() - e1.getY();
      if (Math.abs(dX) > Math.abs(dY) && Math.abs(dX) > SWIPE_DISTANCE_THRESHOLD && Math.abs(velocityX) > SWIPE_VELOCITY_THRESHOLD) {
        if (dX > 0) {
          if (horizontalListener != null)
            return horizontalListener.onSwipeRight();
        }
        else {
          if (horizontalListener != null)
            return horizontalListener.onSwipeLeft();
        }
      }
      else if (Math.abs(dY) > Math.abs(dX) && Math.abs(dY) > SWIPE_DISTANCE_THRESHOLD && Math.abs(velocityY) > SWIPE_VELOCITY_THRESHOLD) {
        if (dY > 0) {
          if (verticalListener != null)
            return verticalListener.onSwipeDown();
        }
        else {
          if (verticalListener != null)
            return verticalListener.onSwipeUp();
        }
      }
      return false;
    }
  }

}

Setting windowAnimationStyle to @null didn't seem to have any effect. I would like to disable the items in the bottom sheet "floating in" at different speeds without disabling the slide in animation on the bottom sheet, to match the Android Lollipop share intent animation and Google Drive's bottom sheet.

Hi,

What's the recommended way to show a checkable menu as a Bottom Sheet? I've tried using the default as described here, and the radio buttons don't show up. Thanks.

Hi, in your sample, when I view a Style from Theme sheet, sheets slide up correctly. Then when I click more, the sheet slides up to full screen. However, after that there is no way to minimize it back down to previous size. Instead, it closes completely when I slide it down.

Please advise, Igor

Previously the navbar height would be 0 for my Oneplus One (Android 7.1), so I interchanged your navBarHeight code for mine. :D Now it gives the correct padding again.

Hey, nice library. I do see something wierd with a transparent navigation bar. On my Nexus7 the sheet does the correct thing and adds padding to the sheet bottom. On my OneplusOne the sheet doesn't do the correct thing and make the sheet overlap with the navigation bar :(

My statusbar height implementation works on all my activities. Could you compare it with yours?

    private static boolean hasNavBar(Context context) {
        // Kitkat and less shows container above nav bar
        if (android.os.Build.VERSION.SDK_INT <= Build.VERSION_CODES.KITKAT) {
            return false;
        }
        // Emulator
        if (Build.FINGERPRINT.startsWith("generic")) {
            return true;
        }
        boolean hasMenuKey = ViewConfiguration.get(context).hasPermanentMenuKey();
        boolean hasBackKey = KeyCharacterMap.deviceHasKey(KeyEvent.KEYCODE_BACK);
        boolean hasNoCapacitiveKeys = !hasMenuKey && !hasBackKey;
        Resources resources = context.getResources();
        int id = resources.getIdentifier("config_showNavigationBar", "bool", "android");
        boolean hasOnScreenNavBar = id > 0 && resources.getBoolean(id);
        return hasOnScreenNavBar || hasNoCapacitiveKeys || getNavigationBarHeight(context, true) > 0;
    }

    public static int getNavigationBarHeight(Context context, boolean skipRequirement) {
        int resourceId = context.getResources().getIdentifier("navigation_bar_height", "dimen", "android");
        if (resourceId > 0 && (skipRequirement || hasNavBar(context))) {
            return context.getResources().getDimensionPixelSize(resourceId);
        }
        return 0;
    }

We're using Material Design icons from the Google library, at 48dp.

When using list style, the icons look great.

When using grid style, the icons are blurry on various tablets, but not on phones. I get the same result whether I build the Bottom sheet using items defined in a menu XML file, or dynamically from code (using enum values).

Overview

Up Close

Should @style/BottomSheet_GridItemImage just use wrap_content for layout_width and layout_height instead of explicitly specifying 48dp?