-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Introduce QuestionMarkTooltip #274
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.
this pr makes things much "class"ier.. !
@@ -3,21 +3,21 @@ import React, {Component, createRef, MouseEvent, RefObject} from 'react' | |||
import classnames from 'classnames' | |||
|
|||
// Components | |||
import {ClickOutside} from '../ClickOutside/ClickOutside' | |||
import {ClickOutside} from '../../ClickOutside/ClickOutside' |
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.
relative addresses scare me a bit
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.
I am looking into setting up absolute paths but I have no idea how to configure builds
Closes #266
Changes
Popover
calledQuestionMarkTooltip
StandardProps
intoStandardClassProps
andStandardFunctionProps
onShow
andonHide
toPopover
Screenshots
Checklist
Check all that apply