Bug 1826091 - Enable Pull to Refresh in release (#1521)

fenix/113.0
Joe Mahon 1 year ago committed by GitHub
parent 31ae668d38
commit e119905271

@ -24,7 +24,7 @@ object FeatureFlags {
* Pull-to-refresh allows you to pull the web content down far enough to have the page to * Pull-to-refresh allows you to pull the web content down far enough to have the page to
* reload. * reload.
*/ */
val pullToRefreshEnabled = Config.channel.isNightlyOrDebug || Config.channel.isBeta const val pullToRefreshEnabled = true
/** /**
* Enables the Sync Addresses feature. * Enables the Sync Addresses feature.

@ -267,18 +267,15 @@ class BrowserFragmentTest {
@Test @Test
fun `WHEN isPullToRefreshEnabledInBrowser is disabled THEN pull down refresh is disabled`() { fun `WHEN isPullToRefreshEnabledInBrowser is disabled THEN pull down refresh is disabled`() {
every { FeatureFlags.pullToRefreshEnabled } returns true
every { context.settings().isPullToRefreshEnabledInBrowser } returns true every { context.settings().isPullToRefreshEnabledInBrowser } returns true
assertTrue(browserFragment.shouldPullToRefreshBeEnabled(false)) assertTrue(browserFragment.shouldPullToRefreshBeEnabled(false))
every { FeatureFlags.pullToRefreshEnabled } returns true
every { context.settings().isPullToRefreshEnabledInBrowser } returns false every { context.settings().isPullToRefreshEnabledInBrowser } returns false
assertTrue(!browserFragment.shouldPullToRefreshBeEnabled(false)) assertTrue(!browserFragment.shouldPullToRefreshBeEnabled(false))
} }
@Test @Test
fun `WHEN in fullscreen THEN pull down refresh is disabled`() { fun `WHEN in fullscreen THEN pull down refresh is disabled`() {
every { FeatureFlags.pullToRefreshEnabled } returns true
every { context.settings().isPullToRefreshEnabledInBrowser } returns true every { context.settings().isPullToRefreshEnabledInBrowser } returns true
assertTrue(browserFragment.shouldPullToRefreshBeEnabled(false)) assertTrue(browserFragment.shouldPullToRefreshBeEnabled(false))
assertTrue(!browserFragment.shouldPullToRefreshBeEnabled(true)) assertTrue(!browserFragment.shouldPullToRefreshBeEnabled(true))

Loading…
Cancel
Save