From 3d71f7481161b85f9451f68b4bcdcd4b6bf47775 Mon Sep 17 00:00:00 2001
From: mhutti1 <mhutti1@gmail.com>
Date: Mon, 6 Feb 2017 19:18:47 +0000
Subject: [PATCH] Fixed drawer size #353

---
 res/layout/drawer_right.xml                                 | 6 ++++++
 src/org/kiwix/kiwixmobile/KiwixMobileActivity.java          | 4 ----
 .../kiwixmobile/views/web/ToolbarScrollingKiwixWebView.java | 2 +-
 3 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/res/layout/drawer_right.xml b/res/layout/drawer_right.xml
index e9c1d2c06..951724eff 100644
--- a/res/layout/drawer_right.xml
+++ b/res/layout/drawer_right.xml
@@ -10,6 +10,12 @@
   android:orientation="vertical"
   android:layout_marginTop="?actionBarSize">
 
+  <View
+    android:id="@+id/view"
+    android:layout_width="fill_parent"
+    android:layout_height="1dp"
+    android:background="?attr/dividerColor"/>
+
   <android.support.v7.widget.RecyclerView
     android:id="@+id/right_drawer_list"
     android:layout_width="match_parent"
diff --git a/src/org/kiwix/kiwixmobile/KiwixMobileActivity.java b/src/org/kiwix/kiwixmobile/KiwixMobileActivity.java
index 6c6691e86..c5ecf5006 100644
--- a/src/org/kiwix/kiwixmobile/KiwixMobileActivity.java
+++ b/src/org/kiwix/kiwixmobile/KiwixMobileActivity.java
@@ -346,10 +346,6 @@ public class KiwixMobileActivity extends AppCompatActivity implements WebViewCal
       }
     });
 
-    // account for status bar height if translucent
-    final int statusBarHeight = DimenUtils.getTranslucentStatusBarHeight(this);
-    tableDrawerRight.setPadding(0, statusBarHeight, 0, 0);
-
     tableDrawerAdapter.notifyDataSetChanged();
 
     tabDrawerAdapter.setTabClickListener(new TabDrawerAdapter.TabClickListener() {
diff --git a/src/org/kiwix/kiwixmobile/views/web/ToolbarScrollingKiwixWebView.java b/src/org/kiwix/kiwixmobile/views/web/ToolbarScrollingKiwixWebView.java
index fd86e3d41..d234e8467 100644
--- a/src/org/kiwix/kiwixmobile/views/web/ToolbarScrollingKiwixWebView.java
+++ b/src/org/kiwix/kiwixmobile/views/web/ToolbarScrollingKiwixWebView.java
@@ -53,7 +53,7 @@ public class ToolbarScrollingKiwixWebView extends KiwixWebView {
     toolbarView.setTranslationY(newTranslation + statusBarHeight);
     this.setTranslationY(newTranslation + toolbarHeight + statusBarHeight);
     if (listener != null && newTranslation != originalTranslation) {
-      if (newTranslation == -toolbarHeight) {
+      if (newTranslation == -toolbarHeight -statusBarHeight) {
         listener.onToolbarHidden();
       } else if (newTranslation == 0) {
         listener.onToolbarDisplayed();