Skip to content

Commit b18e83a

Browse files
committed
Merge branch 'hotfix/5.78.1' into main
2 parents 9276124 + 5637226 commit b18e83a

File tree

2 files changed

+12
-5
lines changed

2 files changed

+12
-5
lines changed

app/src/main/java/com/duckduckgo/app/browser/BrowserActivity.kt

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ import android.content.Context
2121
import android.content.Intent
2222
import android.content.Intent.EXTRA_TEXT
2323
import android.os.Bundle
24+
import android.os.Handler
2425
import android.os.Message
2526
import android.view.View
2627
import android.widget.Toast
@@ -373,7 +374,13 @@ class BrowserActivity : DuckDuckGoActivity(), CoroutineScope by MainScope() {
373374
}
374375

375376
private fun hideMockupOmnibar() {
376-
appBarLayoutMockup.visibility = View.GONE
377+
// Delaying this code to avoid race condition when fragment and activity recreated
378+
Handler().postDelayed(
379+
{
380+
appBarLayoutMockup?.visibility = View.GONE
381+
},
382+
300
383+
)
377384
}
378385

379386
companion object {

app/src/main/java/com/duckduckgo/app/browser/BrowserTabFragment.kt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -482,15 +482,13 @@ class BrowserTabFragment :
482482
appBarLayout.setExpanded(true)
483483
webView?.onPause()
484484
webView?.hide()
485-
omnibarScrolling.disableOmnibarScrolling(toolbarContainer)
486485
homeBackgroundLogo.showLogo()
487486
}
488487

489488
private fun showBrowser() {
490489
newTabLayout.gone()
491490
webView?.show()
492491
webView?.onResume()
493-
omnibarScrolling.enableOmnibarScrolling(toolbarContainer)
494492
homeBackgroundLogo.hideLogo()
495493
}
496494

@@ -1431,12 +1429,14 @@ class BrowserTabFragment :
14311429
fireMenuButton?.isVisible = viewState.fireButton is FireButton.Visible
14321430
menuButton?.isVisible = viewState.showMenuButton
14331431

1432+
// omnibar only scrollable when browser showing and the fire button is not promoted
14341433
if (viewState.fireButton.playPulseAnimation()) {
1435-
appBarLayout.setExpanded(true, true)
14361434
omnibarScrolling.disableOmnibarScrolling(toolbarContainer)
14371435
playPulseAnimation()
14381436
} else {
1439-
omnibarScrolling.enableOmnibarScrolling(toolbarContainer)
1437+
if (viewState.browserShowing) {
1438+
omnibarScrolling.enableOmnibarScrolling(toolbarContainer)
1439+
}
14401440
pulseAnimation.stop()
14411441
}
14421442
}

0 commit comments

Comments
 (0)