-
Notifications
You must be signed in to change notification settings - Fork 95
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 TooltipIcon component to Related Resources #266
Conversation
@@ -276,7 +276,7 @@ | |||
@allowAddingExternalLinks={{true}} | |||
@headerTitle="Related resources" | |||
@modalHeaderTitle="Add related resource" | |||
@modalInputPlaceholder="Paste a URL or search documents..." | |||
@modalInputPlaceholder="Search docs or paste a URL..." |
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.
Reversed the order to be consistent with the tooltip text.
@@ -283,7 +294,7 @@ export default class TooltipModifier extends Modifier<TooltipModifierSignature> | |||
middleware: [ | |||
offset(8), | |||
flip(), | |||
shift(), | |||
shift({ padding: 20 }), |
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.
This prevents the tooltip from hitting the edge of the viewport by 20px.
@@ -163,6 +177,7 @@ module( | |||
@headerTitle="Test title" | |||
@modalHeaderTitle="Add related resource" | |||
@modalInputPlaceholder="Paste a URL or search documents..." | |||
@scrollContainer={{this.body}} |
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.
These lines fix Glint errors in the test.
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.
🎉
Adds a basic TooltipIcon component and implements it on the RelatedResources component.