From cab3ef3f72ef3c9e6797c425a207b3cd043d98b4 Mon Sep 17 00:00:00 2001
From: Chumva <chumva.forever@gmail.com>
Date: Mon, 9 Jul 2018 16:17:39 +0300
Subject: [PATCH] rename changeContinueButtonEnabled and clear resources

---
 .../res/drawable/btn_border_bg.xml            |  2 +-
 OsmAnd-telegram/res/values/strings.xml        |  1 -
 .../osmand/telegram/ui/LoginDialogFragment.kt | 37 ++++++++-----------
 3 files changed, 16 insertions(+), 24 deletions(-)

diff --git a/OsmAnd-telegram/res/drawable/btn_border_bg.xml b/OsmAnd-telegram/res/drawable/btn_border_bg.xml
index ed2c7083c6..e322bda1b2 100644
--- a/OsmAnd-telegram/res/drawable/btn_border_bg.xml
+++ b/OsmAnd-telegram/res/drawable/btn_border_bg.xml
@@ -17,7 +17,7 @@
 		<shape android:shape="rectangle">
 			<solid android:color="@null" />
 			<corners android:radius="@dimen/dialog_button_radius" />
-			<stroke android:width="1dp" android:color="#f0f0f0" />
+			<stroke android:width="1dp" android:color="@color/ctrl_light" />
 		</shape>
 	</item>
 
diff --git a/OsmAnd-telegram/res/values/strings.xml b/OsmAnd-telegram/res/values/strings.xml
index f7aecc81cd..7705646677 100644
--- a/OsmAnd-telegram/res/values/strings.xml
+++ b/OsmAnd-telegram/res/values/strings.xml
@@ -1,5 +1,4 @@
 <resources>
-	<string name="versionFeatures">+play_market -amazon -blackberry</string>
 	<string name="get_telegram_title">Registration in Telegram</string>
 	<string name="get_telegram_account_first">You need an account in Telegram to use the capabilities of OsmAnd Location Sharing.</string>
 	<string name="get_telegram_description_continue">If you want to continue, please install the Telegram from Google Play and register your account.</string>
diff --git a/OsmAnd-telegram/src/net/osmand/telegram/ui/LoginDialogFragment.kt b/OsmAnd-telegram/src/net/osmand/telegram/ui/LoginDialogFragment.kt
index dd13f1dedf..8b51c12d29 100644
--- a/OsmAnd-telegram/src/net/osmand/telegram/ui/LoginDialogFragment.kt
+++ b/OsmAnd-telegram/src/net/osmand/telegram/ui/LoginDialogFragment.kt
@@ -132,10 +132,7 @@ class LoginDialogFragment : DialogFragment() {
 			showWelcomeDialog = args.getBoolean(SHOW_WELCOME_DIALOG_PARAM_KEY, false)
 		}
 		val view = inflater.inflate(R.layout.login_dialog, container)
-		val button = view?.findViewById<Button>(R.id.continue_button)
-		if (button != null) {
-			continueButton = button
-		}
+		continueButton = view.findViewById(R.id.continue_button)
 
 		buildDialog(view)
 		view.viewTreeObserver.addOnGlobalLayoutListener {
@@ -146,27 +143,24 @@ class LoginDialogFragment : DialogFragment() {
 			val softKeyboardVisible = keypadHeight > screenHeight * SOFT_KEYBOARD_MIN_DETECTION_SIZE
 			if (!softKeyboardShown && softKeyboardVisible) {
 				softKeyboardShown = softKeyboardVisible
-				setContinueButtonSize(true)
+				transformContinueButton(true)
 			} else if (softKeyboardShown && !softKeyboardVisible) {
-				setContinueButtonSize(false)
+				transformContinueButton(false)
 			}
 			softKeyboardShown = softKeyboardVisible
 		}
 		return view
 	}
 
-	private fun setContinueButtonSize(isWide: Boolean) {
-		val params = continueButton.layoutParams
-		if (params is ViewGroup.MarginLayoutParams) {
-			if (isWide) {
-				params.setMargins(params.leftMargin, params.topMargin, params.rightMargin, AndroidUtils.dpToPx(context!!, 16F))
-				params.width = ViewGroup.LayoutParams.MATCH_PARENT
-			} else {
-				params.setMargins(params.leftMargin, params.topMargin, params.rightMargin, AndroidUtils.dpToPx(context!!, 40F))
-				params.width = ViewGroup.LayoutParams.WRAP_CONTENT
-			}
-			continueButton.requestLayout()
+	private fun transformContinueButton(expanded: Boolean) {
+		val params = continueButton.layoutParams as ViewGroup.MarginLayoutParams
+		val margin = if (expanded) 16f else 40f
+		val width = if (expanded) ViewGroup.LayoutParams.MATCH_PARENT else ViewGroup.LayoutParams.WRAP_CONTENT
+		params.apply {
+			setMargins(leftMargin, topMargin, rightMargin, AndroidUtils.dpToPx(context!!, margin))
+			this.width = width
 		}
+		continueButton.requestLayout()
 	}
 
 	override fun onCreateDialog(savedInstanceState: Bundle?): Dialog {
@@ -385,11 +379,10 @@ class LoginDialogFragment : DialogFragment() {
 	}
 
 	private fun changeContinueButtonEnabled(enabled: Boolean) {
-		if (enabled) {
-			continueButton.setTextColor(ContextCompat.getColor(context!!, R.color.white))
-		} else {
-			continueButton.setTextColor(ContextCompat.getColor(context!!, R.color.secondary_text_light))
+		if (enabled != continueButton.isEnabled) {
+			val color = if (enabled) R.color.white else R.color.secondary_text_light
+			continueButton.setTextColor(ContextCompat.getColor(context!!, color))
+			continueButton.isEnabled = enabled
 		}
-		continueButton.isEnabled = enabled
 	}
 }