-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Duck.ai: Omnibar interactions #7074
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Duck.ai: Omnibar interactions #7074
Conversation
|
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. This stack of pull requests is managed by Graphite. Learn more about stacking. |
0b7b286 to
a76d661
Compare
22ef7ab to
f125341
Compare
d29475e to
a380796
Compare
6402858 to
cf1afcc
Compare
cf1afcc to
194e53a
Compare
a380796 to
137ee1f
Compare
| 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs updated tests
joshliebe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice one! LGTM 🚀
81873cb
into
feature/david/10-24-duck.ai_omnibar
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

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