Onboarding Brand Design Update: Add in-context SERP dialog

[mikescamell]

Task/Issue URL: https://app.asana.com/1/137249556945/project/1207908166761516/task/1212699268790179?focus=true

Description

Migrates the SERP contextual onboarding dialog ("DuckDuckGo Search!") to the new brand-design layout, gated behind the onboardingBrandDesignUpdate feature flag. Lays the foundation (base class, include layouts, dismiss button styling, banner-animation pipeline, rotation handling) that the remaining contextual CTAs will plug into in follow-up PRs.

With the flag on:

• SERP CTA renders in the new card chrome with title and body as distinct blocks (legacy was a single inline-bolded sentence).
• SERP background banner slides up from behind the card on appear and slides out on dismiss / content swap.
• New circular bordered dismiss button plus a two-layer drop shadow at the bottom edge so the dialog reads as a layer above the browser content.

Out of scope (still legacy / stub-only with the flag on): fire-button, no-trackers, main-network, trackers-blocked, site-suggestions, end. Dax*BrandDesignUpdateContextualCta classes exist as scaffolding for follow-ups.

Why all the rotation-handling code

BrowserActivity declares android:configChanges="keyboardHidden|orientation|screenSize|smallestScreenSize|screenLayout|navigation|keyboard" in AndroidManifest.xml, so Android does not recreate the activity on orientation changes — onConfigurationChanged fires and the existing view tree stays in place. The system never auto-swaps the layout-land variant in for us, so we have to do it manually. That's why this PR introduces:

• Command.ReinflateBrandDesignContextualDialog emitted from the VM on orientation change (only when the brand-design flag is enabled).
• BrowserTabFragmentRenderer.reinflateContextualBrandDesignDialog() which inflates the new orientation's layout, transplants its children into the existing root, and re-shows the CTA via a new instantShow snap path so it lands fully populated (no replay of the typing animation).
• A landscape variant of include_onboarding_in_context_dax_dialog_brand_design_update.xml plus orientation-keyed bools.xml for the options-content height cap.
• Coordination with the animation pipeline: running animators are cancelled before re-inflation, a visibility guard prevents resurrecting a dismissed CTA, and the onCtaShown pixel is gated so it doesn't double-fire on rotation re-inflate.

Other notes

• Dev-settings fix so the bulk "Onboarding Completed" toggle clears DAX_DIALOG_NETWORK / DAX_DIALOG_OTHER flags — without it, iterative testing of the contextual stack silently fails to re-show the in-context site-suggestions dialog after reset.
• cancelRunningAnimations() added to both BrandDesignUpdateBubbleCta (pre-existing, same gap on develop) and the new contextual base class — running fade-in/fade-out AnimatorSets and the container's ViewPropertyAnimator now get cancelled on dismiss so their listeners can't fire on a .gone() view.
• Telemetry parity: pixel names, ctaPixelParam, and ctaId match the legacy DaxSerpCta exactly — no new pixels, no renamed params.

Steps to test this PR

Designs

Please run all testing steps for in-context dialog changes from the contextual-end branch/PR to ease the testing burden

To see these changes patch (Linear onboarding flag included just for continuity)

Index: app/src/main/java/com/duckduckgo/app/onboardingbranddesignupdate/OnboardingBrandDesignUpdateToggles.kt
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/app/src/main/java/com/duckduckgo/app/onboardingbranddesignupdate/OnboardingBrandDesignUpdateToggles.kt b/app/src/main/java/com/duckduckgo/app/onboardingbranddesignupdate/OnboardingBrandDesignUpdateToggles.kt
--- a/app/src/main/java/com/duckduckgo/app/onboardingbranddesignupdate/OnboardingBrandDesignUpdateToggles.kt	(revision 6fd565c8894daba16281ae9bcf3452d038ae8d6d)
+++ b/app/src/main/java/com/duckduckgo/app/onboardingbranddesignupdate/OnboardingBrandDesignUpdateToggles.kt	(date 1777967047929)
@@ -34,13 +34,13 @@
      * Main toggle for the onboarding brand design update feature.
      * Default value: false (disabled).
      */
-    @Toggle.DefaultValue(DefaultFeatureValue.FALSE)
+    @Toggle.DefaultValue(DefaultFeatureValue.TRUE)
     fun self(): Toggle
 
     /**
      * Toggle for the brand design update variant.
      * Default value: false (disabled).
      */
-    @Toggle.DefaultValue(DefaultFeatureValue.FALSE)
+    @Toggle.DefaultValue(DefaultFeatureValue.TRUE)
     fun brandDesignUpdate(): Toggle
 }
Index: app/src/main/java/com/duckduckgo/app/onboarding/ui/OnboardingViewModel.kt
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/app/src/main/java/com/duckduckgo/app/onboarding/ui/OnboardingViewModel.kt b/app/src/main/java/com/duckduckgo/app/onboarding/ui/OnboardingViewModel.kt
--- a/app/src/main/java/com/duckduckgo/app/onboarding/ui/OnboardingViewModel.kt	(revision 6fd565c8894daba16281ae9bcf3452d038ae8d6d)
+++ b/app/src/main/java/com/duckduckgo/app/onboarding/ui/OnboardingViewModel.kt	(date 1777657893440)
@@ -45,7 +45,7 @@
     val viewState = _viewState.asStateFlow()
 
     fun initializePages() {
-        pageLayoutManager.buildPageBlueprints()
+        pageLayoutManager.buildBrandDesignUpdatePageBlueprints()
     }
 
     fun pageCount(): Int {
@@ -69,8 +69,8 @@
         }
     }
 
-    fun initializeOnboardingSkipper() {
-        if (!appBuildConfig.canSkipOnboarding) return
+    fun initializeOnboardingSkipper() { 
+        return
 
         // delay showing skip button until privacy config downloaded
         viewModelScope.launch {

UI changes

See here

---

Note

Medium Risk
Moderate UI/animation changes to the in-browser onboarding CTA pipeline, including new orientation-change reinflation logic that could affect dialog rendering/dismissal on rotation.

Overview
Adds a brand-design variant of the in-context onboarding Dax dialog and wires the SERP CTA to render with the new layout (gated by the onboardingBrandDesignUpdate toggle), including new title/description strings.

Introduces a new OnboardingDaxDialogCta.BrandDesignContextualDaxDialogCta base with coordinated typing/fade animations, tap-to-skip behavior, background banner slide-in/out, and explicit animation cancellation on dismiss; also updates the existing brand-design bubble CTA to cancel running animations.

Updates BrowserTabFragment/BrowserTabViewModel to detect orientation changes and reinflate the brand-design contextual dialog via a new Command.ReinflateBrandDesignContextualDialog, plus new portrait/landscape layouts, shadow/dismiss-button drawables, and onboarding theme attributes (onboardingShadowPurple/onboardingShadowBlue).

Adds scaffolding Dax*BrandDesignUpdateContextualCta classes for other contextual CTAs (mostly stubbed for later), expands onboarding dev settings CTA visibility, and updates/extends unit tests around reinflation and the new animation helpers.

^{Reviewed by Cursor Bugbot for commit 51df44e. Bugbot is set up for automated code reviews on this repo. Configure here.}

[mikescamell]

• Onboarding Brand Design Update: Dismiss intro bubble on submit #8526
• Onboarding Brand Design Update: Update welcome screen copy #8515
• Onboarding Brand Design Update: Add Default Browser page #8514
• Onboarding Brand Design Update: Add in-context End dialog #8484
• Onboarding Brand Design Update: Add in-context Fire Button dialog + Inferno fire animation #8482
• Onboarding Brand Design Update: Add in-context No Trackers dialog #8481
• Onboarding Brand Design Update: Add in-context Main Network dialog #8480
• Onboarding Brand Design Update: Add in-context Trackers Blocked dialog #8479
• Onboarding Brand Design Update: Add in-context Site Suggestions dialog #8483
• Onboarding Brand Design Update: Add in-context SERP dialog #8439 👈 (View in Graphite)
• develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 113d5b1. Configure here.}

[LukasPaczos]

🚢