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

border-radius for tooltips/popovers incorrectly affected by .input-append #6823

Closed
joshdick opened this issue Feb 6, 2013 · 2 comments
Closed

Comments

@joshdick
Copy link

joshdick commented Feb 6, 2013

Elements that trigger tooltips/popovers, as well as child elements in popovers, seem to be incorrectly affected by the input-append class. These elements lose their top-right and bottom-right border-radius when their associated tooltip/popover displays.

Here is a jsFiddle that shows the issue: http://jsfiddle.net/gb2YN/

Reproducible with Bootstrap 2.2.2 in Chrome 24, Firefox 18.0.2, and IE9.

Doesn't appear to have been fixed in 2.3.0-wip yet.

@cvrebert
Copy link
Collaborator

cvrebert commented Feb 6, 2013

Duplicate of #6275?

@mdo
Copy link
Member

mdo commented Feb 7, 2013

This is a dupe a few times over.

With v2.3, you'll need to specify a container to avoid these kinds of issues. This is the best way we could come up with to deal with improved placement of tooltips and popovers without having to endure the clusterfuck of z-index issues we ran into previously.

There's no other way around this via CSS as we're using the + selector, and without specifying a container other than the immediate parent for tooltips and popovers, you effectively nullify that selector. I'll do some work to update the docs in #6346 to reiterate this requirement as it'll likely choke folks up a bit.

@mdo mdo closed this as completed Feb 7, 2013
kasperp referenced this issue in configit/bootstrap Feb 8, 2013
* upstream/master: (162 commits)
  add 2.3 changelog section
  remove prevent default on focus
  only fire hidden once backdrop has been removed
  if no callback, don't try to call callback ya goof
  Fixes #6823: add some docs notes about tooltips in input groups
  unit tests
  dont remove title attribute part 2
  Update link to changelog
  clean up styles
  Fixes #6808: mention IE lame-itude in docs for invalid inputs
  Fixes #6190: Add print utility classes
  screwed up a merge and an old version number crept in
  don't hide if already hidden ya n00b
  clear interval in cycle + rebuild
  update jquery to 1.9
  derp, fix broke css and recompile
  restore comma between selectors
  Fixes #6794: incorrect docs mention of th element
  fix typo
  Add changelog file to repo instead of relying on the wiki
  ...

Conflicts:
	docs/assets/css/bootstrap-responsive.css
	docs/assets/css/bootstrap.css
	docs/assets/js/bootstrap-tooltip.js
	docs/assets/js/bootstrap.js
	docs/assets/js/bootstrap.min.js
	docs/javascript.html
	docs/templates/pages/javascript.mustache
	js/bootstrap-tooltip.js
svda pushed a commit to myxt/bootstrap that referenced this issue Mar 30, 2013
* 'master' of https://github.com/twitter/bootstrap: (292 commits)
  fix pluralization since "a" is in front
  update changelog
  shiiiit
  woops that should stay 2.3.0
  2.3.1
  rebuild
  whitespace
  WHOOPS ( >_>)
  only pass unique options through to child nodes in tooltips
  rebuild
  pass true to cycle
  le sigh
  Backport v3's readme update on npm install to 2.3
  add 2.3 changelog section
  remove prevent default on focus
  only fire hidden once backdrop has been removed
  if no callback, don't try to call callback ya goof
  Fixes twbs#6823: add some docs notes about tooltips in input groups
  unit tests
  dont remove title attribute part 2
  ...
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

No branches or pull requests

3 participants