-
-
Notifications
You must be signed in to change notification settings - Fork 79k
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
tooltips/popovers should choose a different placement instead of running off the page #5032
Comments
do you mean the same as #4687 ? |
yeah solution #1 would be the same |
Hey @ianstormtaylor, Thanks for opening this issue! Unfortunately, it looks like it fails to pass the criteria neccessary for submitting to bootstrap. The following things are currently failing:
For a full list of issue filing guidelines, please refer to the bootstrap issue filing guidelines. thanks! |
figured it was only a matter of time before the dumb logic in those messages came around. there's no need for a jsfiddle in this issue because it's a feature suggestion |
Is this an accepted feature request? Because, I recently implemented this functionality and I can submit a pull request if this is accepted. |
This is something that should be added to the bootstrap for sure. We currently have the exact same problem with placements now. |
@asleepysamurai why don't you just submit a pull request and mention it here? Would be helpful to many who need this basic functionality. |
👍 id try to submit a pull request and see what happens |
Ah, but adding a pull request for such a popular project without knowing if it is required just adds to the workload of the maintainers right??? Anyway, for those who need this, this is my modified tooltip plugin: https://gist.github.com/3972145 If, this feature request is accepted, i'll take some time out to test this a bit more thoroughly and then submit a pull request. Cheers! |
Closing as a dupe issue—just saw this earlier today after reading through the issues. |
two ways this could be done:
The text was updated successfully, but these errors were encountered: