-
Notifications
You must be signed in to change notification settings - Fork 363
AI assistant pane is displayed for supported screens when the AI assistant feature flag is enabled #9591
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
Conversation
|
Let's Go! |
srawlins
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 testing; looks great!
| DevToolsTab get tab => | ||
| DevToolsTab.create(tabName: _tabName, gaPrefix: _gaPrefix); | ||
|
|
||
| static const _tabName = 'AI Assistant'; |
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.
Has this name been vetted by... someone? Not sure who 😅 . But I feel like Product or UX might want to sign off on a branding here.
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.
Since it's behind an experiment flag it can easily be changed (this will not be visible to users until we enable the flag).
Work towards #9574
Currently displays a blank AI assistant tab in the bottom pane for the Inspector and Network panels when AI assistant feature flag is enabled: