Skip to content
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 docs for popover and tooltip hoverable attr #934

Closed

Conversation

openorclose
Copy link
Contributor

What is the purpose of this pull request? (put "X" next to an item, remove the rest)

• [x] Enhancement to an existing feature

Adds docs for MarkBind/vue-strap#114

What is the rationale for this request?

Document the new attribute hoverable available for popovers and tooltips.

What changes did you make? (Give an overview)

Added a new entry to the attributes table, and gave an example on how it works.

Provide some example code that this change will affect:

```
</span>
<span id="output">
<popover content="You can select me!" hoverable>This popover's content can be interacted with.</popover>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm wondering, why can't this be the default/only behavior? Is there any downside to making all popovers hoverable?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left a reason here. It is a small problem, but still tolerable enough to make it the only behavior.

@yamgent
Copy link
Member

yamgent commented Jul 18, 2019

Doesn't seem to be relevant anymore after MarkBind/vue-strap#115 is merged.

@yamgent yamgent closed this Jul 18, 2019
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.

None yet

3 participants