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

css "text-align" affecting some Tooltip position #2992

Closed
epelc opened this issue Nov 18, 2014 · 2 comments
Closed

css "text-align" affecting some Tooltip position #2992

epelc opened this issue Nov 18, 2014 · 2 comments

Comments

@epelc
Copy link

epelc commented Nov 18, 2014

Hello If you have two items with a tooltip and use "text-align: right" on their parent element the second elements tooltips alignment gets messed up. Sometimes the tooltip is also too high above your element and sometimes it is aligned towards the right. To see both you need to resize your screen. Also I think the right alignment also requires a slightly longer tooltip. Here is a plunker http://plnkr.co/edit/SnNuUw43eVbShq578ZkQ?p=preview remember to resize your screen alot to see all the problems with the tooltips on the red buttons.

Version info:
Angular: 1.3.2
Bootstrap: 3.3.1
ui-bootstrap: 0.12.0

@chrisirhc chrisirhc added this to the Purgatory milestone Nov 18, 2014
chrisirhc added a commit to chrisirhc/angular-ui-bootstrap that referenced this issue Nov 19, 2014
The $digest that was removed in 32c470 is actually necessary to get the
tooltip contents rendered. The rendered content is important when the
tooltip is on the right edge. See Plunker in angular-ui#2995:
http://plnkr.co/edit/8GZ5F5zj3Xe6IinXQz74?p=preview

Reverts change in 32c470 (angular-ui#2995) but fixes it in another way.

Fixes angular-ui#2995
Fixes angular-ui#2992
Fixes angular-ui#2951
Closes angular-ui#2996
@FabienDehopre
Copy link

image

<div class="col-md-12 text-right">
    <button type="button" class="btn btn-primary"
            ng-disabled="busy" ng-click="validate()"
            tooltip="Valider le lot actuellement afficher."
            tooltip-placement="left">Valider<i class="fa fa-refresh fa-spin" ng-if="validating"></i></button>
</div>

The text-right class on the div make the tooltip appear right aligned over the button instead of on its left.

@chrisirhc
Copy link
Contributor

@LordDawnhunter , could you check if #2996 resolves your issue?

yoge1 added a commit to jiemakel/recon that referenced this issue Mar 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants