From e202ced4e92d8a1b12c37bc1fc61df561a8e06a5 Mon Sep 17 00:00:00 2001
From: Dmitry Donskoy <donskdmitry@mail.ru>
Date: Thu, 24 Jan 2019 10:56:31 +0300
Subject: [PATCH] [android] Fixed review notes
 https://github.com/mapsme/omim/pull/10238#discussion_r250181767
 https://github.com/mapsme/omim/pull/10238#discussion_r250185662
 https://github.com/mapsme/omim/pull/10238#discussion_r250183076

---
 android/res/layout/fragment_confirmation.xml            | 3 ++-
 android/src/com/mapswithme/maps/dialog/AlertDialog.java | 6 +++---
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/android/res/layout/fragment_confirmation.xml b/android/res/layout/fragment_confirmation.xml
index e7512ac2c9..214fe2dfaf 100644
--- a/android/res/layout/fragment_confirmation.xml
+++ b/android/res/layout/fragment_confirmation.xml
@@ -19,7 +19,7 @@
       android:layout_weight="1"
       android:layout_height="wrap_content"
       android:layout_width="0dp"/>
-    <android.support.v7.widget.AppCompatImageView
+    <ImageView
       android:id="@+id/image"
       android:layout_gravity="center_vertical"
       android:layout_width="@dimen/primary_button_min_height"
@@ -36,6 +36,7 @@
   <Button
     android:id="@+id/accept_btn"
     android:layout_width="wrap_content"
+    android:layout_height="wrap_content"
     android:layout_marginTop="@dimen/margin_base_plus"
     style="@style/MwmWidget.Button.Primary"/>
   <TextView
diff --git a/android/src/com/mapswithme/maps/dialog/AlertDialog.java b/android/src/com/mapswithme/maps/dialog/AlertDialog.java
index 38aa3cec68..12fc2cc908 100644
--- a/android/src/com/mapswithme/maps/dialog/AlertDialog.java
+++ b/android/src/com/mapswithme/maps/dialog/AlertDialog.java
@@ -63,7 +63,7 @@ public class AlertDialog extends BaseMwmDialogFragment
   {
     throw new UnsupportedOperationException("By default, you" +
                                             "shouldn't implement this method." +
-                                            " AlertDialog.Builder will do everything itself. " +
+                                            " AlertDialog.Builder will do everything by itself. " +
                                             "But if you want to use this method, " +
                                             "you'll have to implement it");
   }
@@ -260,7 +260,7 @@ public class AlertDialog extends BaseMwmDialogFragment
     }
 
     @DrawableRes
-    public int getImageResId()
+    int getImageResId()
     {
       return mImageResId;
     }
@@ -280,7 +280,7 @@ public class AlertDialog extends BaseMwmDialogFragment
     }
 
     @NonNull
-    public DialogViewStrategyType getDialogViewStrategyType()
+    DialogViewStrategyType getDialogViewStrategyType()
     {
       return mDialogViewStrategyType;
     }