Adding documentation on <Popover />'s mountNode

[jefffriesen]

The container={mountNode} is a bit mysterious in the popover documentation. I see where it's defined in the docs:

 executeCode: function() {
    var mountNode = this.refs.mount.getDOMNode();

But which node it should reference is hard to decipher. I've tried

  componentDidMount: function() {
    var mountNode = this.refs.button.getDOMNode()
}

With no luck (it's still positioned incorrectly). A little bit of documentation of what this node should refer to and how to access it would be really helpful.

[pieterv]

Hey @jefffriesen, yeah the popover docs definitely need some work :P, i'm happy to help with your specific case in the mean time if you leave some code of what your trying to do. Generally you wouldn't need to set the container prop as it defaults to body which is usually fine for most cases. These are the docs from the bootstrap site which might be all we need?

Appends the tooltip to a specific element. Example: container: 'body'. This option is particularly useful in that it allows you to position the tooltip in the flow of the document near the triggering element - which will prevent the tooltip from floating away from the triggering element during a window resize.

[jefffriesen]

Thanks for the reply and the offer. I had to get it done so I just used the vanilla-React approach (no library) using getInitialState() and setState to show and hide the Popover component. I may go back to using this library at some point.

But as for the documentation, you're right the Bootstrap docs note you added above would be helpful. I still how to retrieve mountNode is a little mysterious, so an example with that would be great.

[taion]

This feature actually looks to be broken right now. The example on the components page with scrolling popovers sets container, but renders the popover in the wrong place:

[taion]

This actually is demonstrated in the components demo, though it's broken due to some weirdness. Do we need additional documentation beyond what's there?

[taion]

Will add further documentation on container with #173.