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

Tooltip/popover doesn't work correctly when text is long and without space #16871

Closed
Hiieu opened this issue Jul 23, 2015 · 14 comments
Closed

Tooltip/popover doesn't work correctly when text is long and without space #16871

Hiieu opened this issue Jul 23, 2015 · 14 comments

Comments

@Hiieu
Copy link

Hiieu commented Jul 23, 2015

http://jsfiddle.net/19jaoah4/

It goes outside the popup.

@twbs-lmvtfy
Copy link

Hi @Hiieu!

You appear to have posted a live example (http://fiddle.jshell.net/p8d23y74/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

  • line 15, column 7: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.0.0
  • line 23, column 7: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.0.0

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!

(Please note that this is a fully automated comment.)

@kkirsche
Copy link
Contributor

@Hiieu Please update to the latest version of Bootstrap. and let us know if the error persists. We do not do support for old versions of Bootstrap.

@cvrebert cvrebert changed the title [Tooltip] Doesn't work correctly when text is long and without space Tooltip/popover doesn't work correctly when text is long and without space Jul 24, 2015
@aelharrak
Copy link

Hello,

This is not a bootstrap issue it's related to the html :)

@jared-christensen
Copy link

Here is example of the fix I am proposing. https://jsfiddle.net/frpeqfmL/

@twbs-lmvtfy
Copy link

Hi @jared-christensen!

You appear to have posted a live example (https://fiddle.jshell.net/frpeqfmL/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

  • line 15, column 7: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.0.0
  • line 23, column 7: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.0.0

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!

(Please note that this is a fully automated comment.)

@cvrebert
Copy link
Collaborator

Refs #14162

@cvrebert
Copy link
Collaborator

cvrebert commented Oct 5, 2015

Still broken in v4: http://jsfiddle.net/z4fck0o2/3/

@fat
Copy link
Member

fat commented Nov 15, 2015

i'm going to tag this as a css issue.

You css nerds just need to use word-break or something 👍 (added to .tooltip)

@fat fat added css and removed js labels Nov 15, 2015
@cvrebert
Copy link
Collaborator

CSS possibilities:

Non-CSS tricks:

@cvrebert
Copy link
Collaborator

@mdo Thoughts on using word-break or similar?

@mdo
Copy link
Member

mdo commented Jan 18, 2016

I'm good with word-wrap: break-word;, but we've got that reset-text mixin in there. Do we remove that property from that mixin, start passing parameters, or dupe the property?

@cvrebert
Copy link
Collaborator

I vote for dropping it from the mixin like we did with font-size.
See #18927.

@cvrebert
Copy link
Collaborator

Fixed in v4 via #18927.

@mdo mdo removed this from the v4.0.0-alpha.3 milestone May 12, 2016
@mdo mdo removed the v4 label May 12, 2016
@mdo
Copy link
Member

mdo commented Sep 5, 2016

Bootstrap 3 is no longer being officially developed or supported.

All work has moved onto our next major release, v4. As such, this issue or pull request is being closed as a "won't fix." For additional help and support, we recommend utilizing our community resources. Thanks for your understanding, and see you on the other side of v4!

<3,
@mdo and team

@mdo mdo closed this as completed Sep 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants