WindowxDeveloper
diff --git a/‎app/build.gradle
Lines changed: 1 addition & 1 deletion b/‎app/build.gradle
Lines changed: 1 addition & 1 deletion
diff --git a/‎app/src/main/java/com/duckduckgo/app/browser/BrowserActivity.kt
Lines changed: 1 addition & 2 deletions b/‎app/src/main/java/com/duckduckgo/app/browser/BrowserActivity.kt
Lines changed: 1 addition & 2 deletions
diff --git a/‎app/src/main/java/com/duckduckgo/app/browser/BrowserTabFragment.kt
Lines changed: 27 additions & 3 deletions b/‎app/src/main/java/com/duckduckgo/app/browser/BrowserTabFragment.kt
Lines changed: 27 additions & 3 deletions
diff --git a/‎app/src/main/res/drawable-h400dp-hdpi/full_logo.png
21.6 KB b/‎app/src/main/res/drawable-h400dp-hdpi/full_logo.png
21.6 KB
diff --git a/‎< D2AE !-- -->app/src/main/res/drawable-h400dp-mdpi/full_logo.png
13.5 KB b/‎< D2AE !-- -->app/src/main/res/drawable-h400dp-mdpi/full_logo.png
13.5 KB
diff --git a/‎app/src/main/res/drawable-h400dp-xhdpi/full_logo.png
30.7 KB b/‎app/src/main/res/drawable-h400dp-xhdpi/full_logo.png
30.7 KB
diff --git a/‎app/src/main/res/drawable-h400dp-xxhdpi/full_logo.png
49.3 KB b/‎app/src/main/res/drawable-h400dp-xxhdpi/full_logo.png
49.3 KB
diff --git a/‎app/src/main/res/drawable-h400dp-xxxhdpi/full_logo.png
69.5 KB b/‎app/src/main/res/drawable-h400dp-xxxhdpi/full_logo.png
69.5 KB
diff --git a/‎app/src/main/res/drawable-hdpi/full_logo.png
-2.4 KB b/‎app/src/main/res/drawable-hdpi/full_logo.png
-2.4 KB
diff --git a/‎app/src/main/res/drawable-mdpi/full_logo.png
-1.3 KB b/‎app/src/main/res/drawable-mdpi/full_logo.png
-1.3 KB
@@ -6,7 +6,7 @@ apply plugin: 'kotlin-kapt'
 apply from: '../versioning.gradle'
 
 ext {
-    VERSION_NAME = "5.0.0"
+    VERSION_NAME = "5.0.1"
     USE_ORCHESTRATOR = project.hasProperty('orchestrator') ? project.property('orchestrator') : false
 }
 
 
@@ -78,10 +78,9 @@ class BrowserActivity : DuckDuckGoActivity() {
     }
 
     private fun openNewTab(tabId: String, userQuery: String? = null) {
-        val previousFragment = supportFragmentManager.findFragmentById(R.id.fragmentContainer) as? BrowserTabFragment
         val fragment = BrowserTabFragment.newInstance(tabId, userQuery)
         val transaction = supportFragmentManager.beginTransaction()
-        if (previousFragment == null) {
+        if (currentTab == null) {
             transaction.replace(R.id.fragmentContainer, fragment, tabId)
         } else {
             transaction.hide(currentTab)
 
@@ -22,6 +22,7 @@ import android.arch.lifecycle.Observer
 import android.arch.lifecycle.ViewModelProviders
 import android.content.Context
 import android.content.Intent
+import android.content.res.Configuration
 import android.net.Uri
 import android.os.Bundle
 import android.support.annotation.StringRes
@@ -280,7 +281,11 @@ class BrowserTabFragment : Fragment(), FindListener {
             omnibarTextInput.setText(viewState.omnibarText)
 
             // ensures caret sits at the end of the query
-            omnibarTextInput.post { omnibarTextInput?.setSelection(omnibarTextInput.text.length) }
+            omnibarTextInput.post {
+                omnibarTextInput?.let {
+                    it.setSelection(it.text.length)
+                }
+            }
             appBarLayout.setExpanded(true, true)
         }
 
@@ -406,7 +411,7 @@ class BrowserTabFragment : Fragment(), FindListener {
                     return@setOnMenuItemClickListener true
                 }
                 R.id.browserPopup -> {
-                    hideKeyboard()
+                    hideKeyboardImmediately()
                     launchPopupMenu()
                     return@setOnMenuItemClickListener true
                 }
@@ -564,6 +569,14 @@ class BrowserTabFragment : Fragment(), FindListener {
         addTextChangedListener(textWatcher)
     }
 
+    private fun hideKeyboardImmediately() {
+        if (!isHidden) {
+            Timber.v("Keyboard now hiding")
+            omnibarTextInput.hideKeyboard()
+            focusDummy.requestFocus()
+        }
+    }
+
     private fun hideKeyboard() {
         if (!isHidden) {
             Timber.v("Keyboard now hiding")
@@ -591,11 +604,22 @@ class BrowserTabFragment : Fragment(), FindListener {
 
     override fun onHiddenChanged(hidden: Boolean) {
         super.onHiddenChanged(hidden)
-        if (!hidden) {
+        if (hidden) {
+            webView?.onPause()
+        } else {
+            webView?.onResume()
             viewModel.onViewVisible()
         }
     }
 
+    /**
+     * We don't destroy the activity on config changes like orientation, so we need to ensure we update resources which might change based on config
+     */
+    override fun onConfigurationChanged(newConfig: Configuration) {
+        super.onConfigurationChanged(newConfig)
+        ddgLogo.setImageResource(R.drawable.full_logo)
+    }
+
     private fun resetTabState() {
         omnibarTextInput.text.clear()
         viewModel.resetView()
Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ apply plugin: 'kotlin-kapt'`
`6`	`6`	`apply from: '../versioning.gradle'`
`7`	`7`
`8`	`8`	`ext {`
`9`		`- VERSION_NAME = "5.0.0"`
	`9`	`+ VERSION_NAME = "5.0.1"`
`10`	`10`	`USE_ORCHESTRATOR = project.hasProperty('orchestrator') ? project.property('orchestrator') : false`
`11`	`11`	`}`
`12`	`12`