Skip to content

split address bar option in onboarding#7621

Merged
aibrahim- merged 3 commits intodevelopfrom
feature/aibrahim/split_option_onboarding
Feb 3, 2026
Merged

split address bar option in onboarding#7621
aibrahim- merged 3 commits intodevelopfrom
feature/aibrahim/split_option_onboarding

Conversation

@aibrahim-
Copy link
Copy Markdown
Contributor

@aibrahim- aibrahim- commented Jan 29, 2026

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

Description

  • Add "Split address bar" option in pre-onboarding
  • Add a toggle sub-feature flag to guard the changes.
  • Update all 3 options icons as requested by Design team.

Steps to test this PR

UI changes

| Before | After |
| |


Note

Medium Risk
Medium risk because it changes pre-onboarding UI flow and the persisted omnibarType selection logic, gated by new remote-config toggles; regressions could affect new-user setup and metrics but is limited in scope.

Overview
Adds a third “Split” address bar position choice to the pre-onboarding welcome flow, updating the UI to a new horizontal toggle layout and driving selection via OmnibarType (top/bottom/split) instead of a boolean.

Gates the split option behind a new remote-config sub-flag splitOmnibarWelcomePage (in addition to splitOmnibar), applies a safe fallback to top if split is selected while disabled, and introduces a new unique pixel m_preonboarding_split_address_bar_selected_unique (wired through AppPixelName, pixel definitions, param-removal, and updated tests).

Written by Cursor Bugbot for commit 6a34b9f. This will update automatically on new commits. Configure here.

Copy link
Copy Markdown
Contributor Author

aibrahim- commented Jan 29, 2026

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

@aibrahim- aibrahim- marked this pull request as ready for review January 29, 2026 15:01
Comment thread .maestro/preonboarding/preonboarding_address_bar_split_option_visible.yaml Outdated
Comment thread app/fdroid/release/output-metadata.json Outdated
@aibrahim- aibrahim- force-pushed the feature/aibrahim/split_option_onboarding branch 4 times, most recently from ceb10b8 to b500a2e Compare January 29, 2026 15:36
@github-actions
Copy link
Copy Markdown
Contributor

Privacy Review task: https://app.asana.com/0/69071770703008/1213029344891220

@aibrahim- aibrahim- changed the title split option onboarding split address bar option in onboarding Jan 29, 2026
@aibrahim- aibrahim- requested a review from joshliebe January 30, 2026 11:20
@aibrahim- aibrahim- force-pushed the feature/aibrahim/split_option_onboarding branch 3 times, most recently from dc728af to 36e7cea Compare January 30, 2026 12:47
@aibrahim- aibrahim- force-pushed the feature/aibrahim/split_option_onboarding branch from 36e7cea to ac2a867 Compare January 30, 2026 13:16
@aibrahim- aibrahim- force-pushed the feature/aibrahim/split_option_onboarding branch from de05be5 to d1c8515 Compare February 2, 2026 10:21
Comment thread app/src/main/java/com/duckduckgo/app/onboarding/ui/page/WelcomePageViewModel.kt Outdated
Comment thread app/src/main/java/com/duckduckgo/app/pixels/AppPixelName.kt
Comment thread app/src/main/java/com/duckduckgo/app/onboarding/ui/page/WelcomePage.kt Outdated
Copy link
Copy Markdown
Contributor

@joshliebe joshliebe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The pixel needs to strip atb, once that’s fixed I’ll approve.

Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@joshliebe joshliebe self-assigned this Feb 3, 2026
Copy link
Copy Markdown
Contributor

@joshliebe joshliebe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM! 🚀

@aibrahim- aibrahim- merged commit da848a1 into develop Feb 3, 2026
17 of 26 checks passed
@aibrahim- aibrahim- deleted the feature/aibrahim/split_option_onboarding branch February 3, 2026 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants