Skip to content

fix(tooltip): fix when toggle is not provided#182

Merged
eddywashere merged 1 commit intoreactstrap:masterfrom
TheSharpieOne:fix/tooltip-toggle
Oct 18, 2016
Merged

fix(tooltip): fix when toggle is not provided#182
eddywashere merged 1 commit intoreactstrap:masterfrom
TheSharpieOne:fix/tooltip-toggle

Conversation

@TheSharpieOne
Copy link
Copy Markdown
Member

Before, when props.toggle was not provided, this.toggle would throw
Essentially when someone just provides the isOpen prop and wants
to manually control the hiding/showing.

onMouseOverTooltip would trigger show, which would trigger toggle when the tooltip was not being shown. Without toggle prop, it would error.

Before, when props.toggle was not provided, this.toggle would throw
Essentially when someone just provides the `isOpen` prop and wants
to manaully control the hiding/showing.
@ronnyhaase
Copy link
Copy Markdown
Contributor

👍

@eddywashere
Copy link
Copy Markdown
Member

Should this be added to other components?

@eddywashere eddywashere merged commit 47239f3 into reactstrap:master Oct 18, 2016
@TheSharpieOne TheSharpieOne deleted the fix/tooltip-toggle branch October 18, 2016 03:22
@TheSharpieOne
Copy link
Copy Markdown
Member Author

Which other components? I looked at pop over and it did not work the same way. I mean, we could default some of the callback functions, but at a certain point, they are not explicitly used like it was here.

@eddywashere
Copy link
Copy Markdown
Member

That's right! Nothing to see here :)

On Mon, Oct 17, 2016 at 8:24 PM Evan Sharp [email protected] wrote:

Which other components? I looked at pop over and it did not work the same
way. I mean, we could default some of the callback functions, but at a
certain point, they are not explicitly used like it was here.


You are receiving this because you modified the open/close state.

Reply to this email directly, view it on GitHub
#182 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAYZoMOEnxp-9Q6DO5plw_513-x5oPM2ks5q1DvhgaJpZM4KZB2x
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants