Skip to content

Commit 3244596

Browse files
Merge pull request #4 from mschlitz-trux/position-tooltip-fixes
Position tooltip fixes
2 parents 311c4e8 + d3a6f43 commit 3244596

File tree

2 files changed

+16
-12
lines changed

2 files changed

+16
-12
lines changed

dist/ui-bootstrap-tpls.js

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5158,14 +5158,16 @@ angular.module('ui.bootstrap.tooltip', ['ui.bootstrap.position', 'ui.bootstrap.s
51585158
if (!positionTimeout) {
51595159
positionTimeout = $timeout(function() {
51605160
var placementClasses = $position.parsePlacement(ttScope.placement);
5161-
var ttPosition = $position.positionElements(element, tooltip, ttScope.placement, appendToBody || appendTo, true);
5162-
var placement = ttPosition.placement;
5163-
5164-
// need to add classes prior to placement to allow correct tooltip width calculations
5161+
5162+
// need to add classes prior to placement to allow correct tooltip size calculations
51655163
if (!tooltip.hasClass(placementClasses[0])) {
51665164
tooltip.removeClass(lastPlacement.split('-')[0]);
5167-
tooltip.addClass(placementClasses[0]);
5165+
tooltip.addClass(options.placementClassPrefix + placementClasses[0]);
51685166
}
5167+
5168+
// calculate proper positioning based on available space
5169+
var ttPosition = $position.positionElements(element, tooltip, ttScope.placement, appendToBody || appendTo, true);
5170+
var placement = ttPosition.placement;
51695171

51705172
if (!tooltip.hasClass(options.placementClassPrefix + placement)) {
51715173
tooltip.removeClass(options.placementClassPrefix + lastPlacement);
@@ -5685,7 +5687,7 @@ function ($animate, $sce, $compile, $templateRequest) {
56855687
// // There are no top-left etc... classes
56865688
// // in TWBS, so we need the primary position.
56875689
var position = $uibPosition.parsePlacement(scope.placement);
5688-
element.addClass(position[0]);
5690+
element.addClass('bs-tooltip-' + position[0]);
56895691
}
56905692

56915693
if (scope.popupClass) {

src/tooltip/tooltip.js

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -163,14 +163,16 @@ angular.module('ui.bootstrap.tooltip', ['ui.bootstrap.position', 'ui.bootstrap.s
163163
if (!positionTimeout) {
164164
positionTimeout = $timeout(function() {
165165
var placementClasses = $position.parsePlacement(ttScope.placement);
166-
var ttPosition = $position.positionElements(element, tooltip, ttScope.placement, appendToBody || appendTo, true);
167-
var placement = ttPosition.placement;
168-
169-
// need to add classes prior to placement to allow correct tooltip width calculations
166+
167+
// need to add classes prior to placement to allow correct tooltip size calculations
170168
if (!tooltip.hasClass(placementClasses[0])) {
171169
tooltip.removeClass(lastPlacement.split('-')[0]);
172-
tooltip.addClass(placementClasses[0]);
170+
tooltip.addClass(options.placementClassPrefix + placementClasses[0]);
173171
}
172+
173+
// calculate proper positioning based on available space
174+
var ttPosition = $position.positionElements(element, tooltip, ttScope.placement, appendToBody || appendTo, true);
175+
var placement = ttPosition.placement;
174176

175177
if (!tooltip.hasClass(options.placementClassPrefix + placement)) {
176178
tooltip.removeClass(options.placementClassPrefix + lastPlacement);
@@ -690,7 +692,7 @@ function ($animate, $sce, $compile, $templateRequest) {
690692
// // There are no top-left etc... classes
691693
// // in TWBS, so we need the primary position.
692694
var position = $uibPosition.parsePlacement(scope.placement);
693-
element.addClass(position[0]);
695+
element.addClass('bs-tooltip-' + position[0]);
694696
}
695697

696698
if (scope.popupClass) {

0 commit comments

Comments
 (0)