Skip to content

Conversation

@malmstein
Copy link
Contributor

@malmstein malmstein commented Nov 6, 2025

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

Description

This PR adds CTA for the main actions in the Duck.ai Omnibar

Steps to test this PR

Feature Flag enabled

  • Ensure Fullscreen mode is enabled in AI Features
  • Open Duck.ai
  • Verify new Omnibar is visible
  • Tap on the Fire Button
  • Verify the Bottom Sheet appears (Tab restoration is not implemented, so you’ll see NTP upon restart)
  • Tap on Tab Switcher
  • Verify Tab Manager appears (Duck.ai won’t be shown as a tab, not implemented yet)
  • Tap on the input field
  • Verify Input Screen appears
  • Enter a search query
  • Verify that the browser replaces Duck.ai

Copy link
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

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

@malmstein malmstein mentioned this pull request Nov 6, 2025
8 tasks
@malmstein malmstein force-pushed the feature/david/10-24-duck.ai_omnibar branch from 0b7b286 to a76d661 Compare November 7, 2025 12:01
@malmstein malmstein force-pushed the feature/david/11-06-duck.ai_omnibar_interactions branch from 22ef7ab to f125341 Compare November 7, 2025 12:01
@malmstein malmstein marked this pull request as ready for review November 7, 2025 12:16
@malmstein malmstein force-pushed the feature/david/11-06-duck.ai_omnibar_interactions branch 3 times, most recently from d29475e to a380796 Compare November 7, 2025 15:02
@malmstein malmstein force-pushed the feature/david/10-24-duck.ai_omnibar branch from 6402858 to cf1afcc Compare November 7, 2025 15:02
@malmstein malmstein force-pushed the feature/david/10-24-duck.ai_omnibar branch from cf1afcc to 194e53a Compare November 7, 2025 15:03
@malmstein malmstein force-pushed the feature/david/11-06-duck.ai_omnibar_interactions branch from a380796 to 137ee1f Compare November 7, 2025 15:03
val hasPendingDuckAiOpen = externalIntentProcessingState.hasPendingDuckAiOpen.value
if (!hasPendingTabLaunch && !hasPendingDuckAiOpen) {
// whenever an event fires, so the user switched to a new tab page, launch the input screen
command.value = LaunchInputScreen
Copy link
Contributor

Choose a reason for hiding this comment

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

Needs updated tests

Copy link
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.

Nice one! LGTM 🚀

@malmstein malmstein merged commit 81873cb into feature/david/10-24-duck.ai_omnibar Nov 7, 2025
5 of 6 checks passed
@malmstein malmstein deleted the feature/david/11-06-duck.ai_omnibar_interactions branch November 7, 2025 21:38
malmstein added a commit that referenced this pull request Nov 7, 2025
Task/Issue URL:
https://app.asana.com/1/137249556945/project/1174433894299346/task/1211877638725902?focus=true

### Description
This PR adds CTA for the main actions in the Duck.ai Omnibar

### Steps to test this PR

_Feature Flag enabled_
- [x] Ensure Fullscreen mode is enabled in AI Features
- [x] Open Duck.ai
- [x] Verify new Omnibar is visible
- [x] Tap on the Fire Button
- [x] Verify the Bottom Sheet appears (Tab restoration is not
implemented, so you’ll see NTP upon restart)
- [x] Tap on Tab Switcher
- [x] Verify Tab Manager appears (Duck.ai won’t be shown as a tab, not
implemented yet)
- [x] Tap on the input field
- [x] Verify Input Screen appears
- [x] Enter a search query
- [x] Verify that the browser replaces Duck.ai
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants