Skip to content

Commit 314a226

Browse files
authored
Revert "SplashScreen Animation: Add wink animation (#5565)" (#5666)
Task/Issue URL: https://app.asana.com/0/1207908166761516/1209441074191255 ### Description This reverts commit 1af0d49. There have been some Splashscreen issues on Android 12L so we're reverting and going to try and fix for the next release ### Steps to test this PR _Splashscreen_ - [ ] Open app on a number of different SDK versions, a good sample is 26, 32, 35 - [ ] Check the static logo shows and the app launches (remember there is a bug if you launch the app from Android Studio on to an Android 12L device where you only see the logo if you launch from the home screen) ### UI changes Back to previous static logo
1 parent a90a3fd commit 314a226

File tree

9 files changed

+2
-12034
lines changed

9 files changed

+2
-12034
lines changed

app/src/main/java/com/duckduckgo/app/global/api/PixelParamRemovalInterceptor.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,6 @@ object PixelInterceptorPixelsRequiringDataCleaning : PixelParamRemovalPlugin {
101101
AppPixelName.SET_AS_DEFAULT_PROMPT_CLICK.pixelName to PixelParameter.removeAll(),
102102
AppPixelName.SET_AS_DEFAULT_PROMPT_DISMISSED.pixelName to PixelParameter.removeAll(),
103103
AppPixelName.SET_AS_DEFAULT_IN_MENU_CLICK.pixelName to PixelParameter.removeAll(),
104-
AppPixelName.SPLASHSCREEN_SHOWN.pixelName to PixelParameter.removeAll(),
105104
AppPixelName.MENU_ACTION_NEW_TAB_PRESSED_FROM_SITE.pixelName to PixelParameter.removeAll(),
106105
AppPixelName.MENU_ACTION_NEW_TAB_PRESSED_FROM_SERP.pixelName to PixelParameter.removeAll(),
107106
)

app/src/main/java/com/duckduckgo/app/launch/LaunchBridgeActivity.kt

Lines changed: 2 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,6 @@ import com.duckduckgo.app.browser.R
2525
import com.duckduckgo.app.onboarding.ui.OnboardingActivity
2626
import com.duckduckgo.common.ui.DuckDuckGoActivity
2727
import com.duckduckgo.di.scopes.ActivityScope
28-
import java.time.Instant
29-
import java.time.temporal.ChronoUnit
30-
import kotlinx.coroutines.delay
3128
import kotlinx.coroutines.launch
3229

3330
@InjectWith(ActivityScope::class)
@@ -38,25 +35,13 @@ class LaunchBridgeActivity : DuckDuckGoActivity() {
3835
override fun onCreate(savedInstanceState: Bundle?) {
3936
val splashScreen = installSplashScreen()
4037
super.onCreate(savedInstanceState)
38+
splashScreen.setKeepOnScreenCondition { true }
4139

4240
setContentView(R.layout.activity_launch)
4341

4442
configureObservers()
4543

46-
splashScreen.setOnExitAnimationListener { splashScreenView ->
47-
val splashScreenAnimationEndTime =
48-
Instant.ofEpochMilli(splashScreenView.iconAnimationStartMillis + splashScreenView.iconAnimationDurationMillis)
49-
val remainingAnimationTime = Instant.now().until(
50-
splashScreenAnimationEndTime,
51-
ChronoUnit.MILLIS,
52-
)
53-
54-
lifecycleScope.launch {
55-
viewModel.sendWelcomeScreenPixel()
56-
delay(remainingAnimationTime)
57-
viewModel.determineViewToShow()
58-
}
59-
}
44+
lifecycleScope.launch { viewModel.determineViewToShow() }
6045
}
6146

6247
private fun configureObservers() {

app/src/main/java/com/duckduckgo/app/launch/LaunchViewModel.kt

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,8 @@ import androidx.lifecycle.ViewModel
2020
import com.duckduckgo.anvil.annotations.ContributesViewModel
2121
import com.duckduckgo.app.onboarding.store.UserStageStore
2222
import com.duckduckgo.app.onboarding.store.isNewUser
23-
import com.duckduckgo.app.pixels.AppPixelName.SPLASHSCREEN_SHOWN
2423
import com.duckduckgo.app.referral.AppInstallationReferrerStateListener
2524
import com.duckduckgo.app.referral.AppInstallationReferrerStateListener.Companion.MAX_REFERRER_WAIT_TIME_MS
26-
import com.duckduckgo.app.statistics.pixels.Pixel
2725
import com.duckduckgo.common.utils.SingleLiveEvent
2826
import com.duckduckgo.di.scopes.ActivityScope
2927
import javax.inject.Inject
@@ -34,7 +32,6 @@ import timber.log.Timber
3432
class LaunchViewModel @Inject constructor(
3533
private val userStageStore: UserStageStore,
3634
private val appReferrerStateListener: AppInstallationReferrerStateListener,
37-
private val pixel: Pixel,
3835
) :
3936
ViewModel() {
4037

@@ -45,10 +42,6 @@ class LaunchViewModel @Inject constructor(
4542
data class Home(val replaceExistingSearch: Boolean = false) : Command()
4643
}
4744

48-
fun sendWelcomeScreenPixel() {
49-
pixel.fire(SPLASHSCREEN_SHOWN)
50-
}
51-
5245
suspend fun determineViewToShow() {
5346
waitForReferrerData()
5447

app/src/main/java/com/duckduckgo/app/pixels/AppPixelName.kt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,6 @@ enum class AppPixelName(override val pixelName: String) : Pixel.PixelName {
3737
BROKEN_SITE_ALLOWLIST_REMOVE("m_broken_site_allowlist_remove"),
3838
PROTECTION_TOGGLE_BROKEN_SITE_REPORT("m_protection-toggled-off-breakage-report"),
3939

40-
SPLASHSCREEN_SHOWN("m_splashscreen_shown"),
41-
4240
PREONBOARDING_INTRO_SHOWN_UNIQUE("m_preonboarding_intro_shown_unique"),
4341
PREONBOARDING_COMPARISON_CHART_SHOWN_UNIQUE("m_preonboarding_comparison_chart_shown_unique"),
4442
PREONBOARDING_CHOOSE_BROWSER_PRESSED("m_preonboarding_choose_browser_pressed"),

app/src/test/java/com/duckduckgo/app/launch/LaunchViewModelTest.kt

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -22,13 +22,10 @@ import com.duckduckgo.app.launch.LaunchViewModel.Command.Home
2222
import com.duckduckgo.app.launch.LaunchViewModel.Command.Onboarding
2323
import com.duckduckgo.app.onboarding.store.AppStage
2424
import com.duckduckgo.app.onboarding.store.UserStageStore
25-
import com.duckduckgo.app.pixels.AppPixelName
2625
import com.duckduckgo.app.referral.StubAppReferrerFoundStateListener
2726
import com.duckduckgo.common.test.CoroutineTestRule
28-
import com.duckduckgo.fakes.FakePixel
2927
import kotlinx.coroutines.test.runTest
3028
import org.junit.After
31-
import org.junit.Assert.assertEquals
3229
import org.junit.Rule
3330
import org.junit.Test
3431
import org.mockito.kotlin.any
@@ -48,8 +45,6 @@ class LaunchViewModelTest {
4845
private val userStageStore = mock<UserStageStore>()
4946
private val mockCommandObserver: Observer<LaunchViewModel.Command> = mock()
5047

51-
private val fakePixel: FakePixel = FakePixel()
52-
5348
private lateinit var testee: LaunchViewModel
5449

5550
@After
@@ -62,7 +57,6 @@ class LaunchViewModelTest {
6257
testee = LaunchViewModel(
6358
userStageStore,
6459
StubAppReferrerFoundStateListener("xx"),
65-
fakePixel,
6660
)
6761
whenever(userStageStore.getUserAppStage()).thenReturn(AppStage.NEW)
6862
testee.command.observeForever(mockCommandObserver)
@@ -77,7 +71,6 @@ class LaunchViewModelTest {
7771
testee = LaunchViewModel(
7872
userStageStore,
7973
StubAppReferrerFoundStateListener("xx", mockDelayMs = 1_000),
80-
fakePixel,
8174
)
8275
whenever(userStageStore.getUserAppStage()).thenReturn(AppStage.NEW)
8376
testee.command.observeForever(mockCommandObserver)
@@ -92,7 +85,6 @@ class LaunchViewModelTest {
9285
testee = LaunchViewModel(
9386
userStageStore,
9487
StubAppReferrerFoundStateListener("xx", mockDelayMs = Long.MAX_VALUE),
95-
fakePixel,
9688
)
9789
whenever(userStageStore.getUserAppStage()).thenReturn(AppStage.NEW)
9890
testee.command.observeForever(mockCommandObserver)
@@ -107,7 +99,6 @@ class LaunchViewModelTest {
10799
testee = LaunchViewModel(
108100
userStageStore,
109101
StubAppReferrerFoundStateListener("xx"),
110-
fakePixel,
111102
)
112103
whenever(userStageStore.getUserAppStage()).thenReturn(AppStage.DAX_ONBOARDING)
113104
testee.command.observeForever(mockCommandObserver)
@@ -120,7 +111,6 @@ class LaunchViewModelTest {
120111
testee = LaunchViewModel(
121112
userStageStore,
122113
StubAppReferrerFoundStateListener("xx", mockDelayMs = 1_000),
123-
fakePixel,
124114
)
125115
whenever(userStageStore.getUserAppStage()).thenReturn(AppStage.DAX_ONBOARDING)
126116
testee.command.observeForever(mockCommandObserver)
@@ -133,24 +123,10 @@ class LaunchViewModelTest {
133123
testee = LaunchViewModel(
134124
userStageStore,
135125
StubAppReferrerFoundStateListener("xx", mockDelayMs = Long.MAX_VALUE),
136-
fakePixel,
137126
)
138127
whenever(userStageStore.getUserAppStage()).thenReturn(AppStage.DAX_ONBOARDING)
139128
testee.command.observeForever(mockCommandObserver)
140129
testee.determineViewToShow()
141130
verify(mockCommandObserver).onChanged(any<Home>())
142131
}
143-
144-
@Test
145-
fun whenSendingWelcomeScreenPixelThenSplashScreenShownPixelIsSent() = runTest {
146-
testee = LaunchViewModel(
147-
userStageStore,
148-
StubAppReferrerFoundStateListener("xx", mockDelayMs = Long.MAX_VALUE),
149-
fakePixel,
150-
)
151-
152-
testee.sendWelcomeScreenPixel()
153-
154-
assertEquals(AppPixelName.SPLASHSCREEN_SHOWN.pixelName, fakePixel.firedPixels.first())
155-
}
156132
}

0 commit comments

Comments
 (0)