-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add AI Toolbar visibility toggle in Dashboard #2268
Conversation
Bundle Size Diff
|
Plugin build for 922a821 is ready 🛎️!
|
E2E TestsPlaywright Test Status: Performance ResultsserverResponse: 235.05, firstPaint: 169.9, domContentLoaded: 1560.35, loaded: 1561, firstContentfulPaint: 3437.2, firstBlock: 6653.95, type: 10.98, minType: 10.09, maxType: 12.61, typeContainer: 7.77, minTypeContainer: 6.72, maxTypeContainer: 9.5, focus: 30.12, minFocus: 26.51, maxFocus: 36.84, inserterOpen: 23.85, minInserterOpen: 20.94, maxInserterOpen: 28, inserterSearch: 0.73, minInserterSearch: 0.63, maxInserterSearch: 0.95, inserterHover: 3.05, minInserterHover: 2.6, maxInserterHover: 4.25, listViewOpen: 147.59, minListViewOpen: 129.14, maxListViewOpen: 175.01 |
Everything looks good! Love that you have a test for the feature. 🐛🥳 My only request is can you merge all the commits under |
085993e
to
2b76b97
Compare
Looks good! Thanks @Soare-Robert-Daniel! |
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.
LGTM. Just one nit is that we can move the localized module flag from aiToolbarEnabled to hasModule that already contains other.
2b76b97
to
922a821
Compare
🎉 This PR is included in version 3.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Closes https://github.com/Codeinwp/otter-internals/issues/206
Summary
Added a toggle for the AI Toolbar similar to the CSS module.
Screenshots
Test instructions
Checklist before the final review