Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

chore(button): add known issue for tooltip/popover #5890

Closed
wants to merge 1 commit into from

Conversation

icfantv
Copy link
Contributor

@icfantv icfantv commented May 10, 2016

  • add known issue to button docs indicating a CSS issue when using a tooltip or
    a popover.


### Known issues

To use tooltips or popovers on a button group, make sure you add the applicable tooltip or popover directive directly to your `btn-group` element and set `appendToBody` to `true`. This is due to Bootstrap CSS styling. See [here](http://getbootstrap.com/components/#btn-groups) for more information.
Copy link
Contributor

Choose a reason for hiding this comment

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

The append-to-body is only necessary if adding the tooltip to an element inside the btn-group.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, ok. I wasn't sure I understood this correctly.

@icfantv
Copy link
Contributor Author

icfantv commented May 10, 2016

@RobJacobs, fixed. look good?

@RobJacobs
Copy link
Contributor

@icfantv I would remove the mention of adding the tooltip to the btn-group, so maybe something like:

make sure you set appendToBody to true

@icfantv
Copy link
Contributor Author

icfantv commented May 11, 2016

@RobJacobs ok, done.

* add known issue to button docs indicating a CSS issue when using a tooltip or
  a popover.
@wesleycho wesleycho added this to the 1.3.3 milestone May 16, 2016
@wesleycho wesleycho closed this in aec5304 May 16, 2016
@icfantv icfantv deleted the doc_update branch July 21, 2016 15:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants