BottomNavigationView scroll, PopupMenu height, NestedScrollView crash fix

Author: Yanndroid

app/src/main/java/de/dlyt/yanndroid/oneuiexample/MainActivity.java

@@ -156,7 +156,7 @@ public void onClick(View view) {
                 });
 
         //BottomNavigationLayout
-        Drawable icon = getDrawable(R.drawable.ic_samsung_more);
+        Drawable icon = getDrawable(R.drawable.ic_samsung_drawer);
         icon.setColorFilter(getResources().getColor(R.color.sesl_tablayout_text_color), PorterDuff.Mode.SRC_IN);
         for (String s : mTabsTitleName) {
             bnvLayout.addTab(bnvLayout.newTab().setText(s));
@@ -342,13 +342,12 @@ public void onClick(View v) {
     private void popupView(View view) {
         PopupMenu popupMenu = new PopupMenu(view);
         ArrayList<String> list = new ArrayList<>();
-        for (int i = 0; i < 4; i++) list.add("Menu Item " + i);
+        for (int i = 0; i < 24; i++) list.add("Menu Item " + i);
         popupMenu.inflate(list);
         popupMenu.setOnMenuItemClickListener((parent, view1, position, id) -> {
             popupMenu.dismiss();
         });
-        // temp
-        popupMenu.show(-120, (int) ((-bnvLayout.getHeight() * 5) * 0.88f));
+        popupMenu.show();
     }
 
 

yanndroid/oneui/src/main/java/de/dlyt/yanndroid/oneui/view/BottomNavigationView.java

@@ -102,6 +102,9 @@ private void invalidateTabLayout() {
             tabTextWidthSum += width;
             ViewSupport.setPointerIcon(tabView, 1000 /* PointerIcon.TYPE_ARROW */);
         }
+        if (tabTextWidthSum >= getContext().getResources().getDisplayMetrics().widthPixels) {
+            setTabMode(0);
+        }
         addTabPaddingValue(tabTextWidthList, tabTextWidthSum);
     }
 

yanndroid/oneui/src/main/java/de/dlyt/yanndroid/oneui/view/NestedScrollView.java

@@ -1150,7 +1150,7 @@ public boolean onTouchEvent(MotionEvent ev) {
                     if (canOverscroll) {
                         deltaY -= mScrollConsumed[1];
                         final int pulledToY = oldY + deltaY;
-                        if (pulledToY < 0) {
+                        if (mEdgeGlowTop != null && pulledToY < 0) {
                             EdgeEffectCompat.onPull(mEdgeGlowTop, (float) -deltaY / getHeight(), ev.getX(activePointerIndex) / getWidth());
                             if (!mEdgeGlowBottom.isFinished()) {
                                 mEdgeGlowBottom.onRelease();

yanndroid/oneui/src/main/java/de/dlyt/yanndroid/oneui/view/PopupMenu.java

@@ -7,6 +7,7 @@
 import android.widget.AdapterView;
 import android.widget.ArrayAdapter;
 import android.widget.LinearLayout;
+import android.widget.ListAdapter;
 import android.widget.PopupWindow;
 import android.widget.TextView;
 
@@ -45,9 +46,13 @@ public void inflate(ArrayList<String> menu) {
         listView.setDivider(null);
         listView.setSelector(R.drawable.sesl_list_selector);
 
+        listView.measure(ViewGroup.LayoutParams.WRAP_CONTENT, ViewGroup.LayoutParams.WRAP_CONTENT);
+        int height = listView.getMeasuredHeight() + context.getResources().getDimensionPixelSize(R.dimen.sesl_popup_menu_item_bottom_padding) -5;
+
         popupWindow = new PopupWindow(listView);
         popupWindow.setWidth(getPopupMenuWidth());
-        popupWindow.setHeight(ViewGroup.LayoutParams.WRAP_CONTENT);
+        popupWindow.setHeight(height);
+        //popupWindow.setHeight(ViewGroup.LayoutParams.WRAP_CONTENT);
         popupWindow.setAnimationStyle(R.style.MenuPopupAnimStyle);
         popupWindow.setBackgroundDrawable(context.getDrawable(R.drawable.sesl_menu_popup_background));
         popupWindow.setOutsideTouchable(true);