-
Notifications
You must be signed in to change notification settings - Fork 238
feat(compass-assistant): add GuideCue support to DrawerSection COMPASS-9795 #7347
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
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.
Pull Request Overview
This PR adds GuideCue support to DrawerSection components to display guided tooltips for the MongoDB Assistant drawer. The implementation allows drawer sections to show contextual onboarding hints when they first appear.
- Add
appName
prop toCompassAssistantDrawerWithConnections
component for app-specific messaging - Enhance
GuideCue
component to support external trigger nodes viatriggerNode
prop - Implement GuideCue rendering in
DrawerAnchor
with DOM node lookup for drawer buttons
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
packages/compass/src/app/components/workspace.tsx | Pass "Compass" app name to assistant drawer |
packages/compass/src/app/components/compass-assistant-drawer.tsx | Add appName prop and pass through to CompassAssistantDrawer |
packages/compass-web/src/entrypoint.tsx | Pass "Data Explorer" app name to assistant drawer |
packages/compass-web/src/compass-assistant-drawer.tsx | Add appName prop and pass through to CompassAssistantDrawer |
packages/compass-components/src/components/guide-cue/guide-cue.tsx | Add optional triggerNode prop and make trigger prop optional |
packages/compass-components/src/components/drawer-portal.tsx | Implement GuideCue support with DOM lookup and rendering logic |
packages/compass-components/src/components/drawer-portal.spec.tsx | Add test for GuideCue rendering functionality |
packages/compass-assistant/src/compass-assistant-provider.spec.tsx | Update test to include required appName prop |
packages/compass-assistant/src/compass-assistant-drawer.tsx | Add appName prop and GuideCue configuration for assistant drawer |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
f9eb82f
to
4f0afe3
Compare
This is a temporary hack until we get better support for targeting the toolbar icon in leafygreen. The overall interface on should survive and the dom element lookups should hopefully go away.
To test locally you can run this in the console and then cmd-shift-r to reload the app: