Fix tooltip position when it's on the bottom - #635

0944f4a4 · Masayuki Tanaka · 78a21e78 · 0944f4a4 · 0944f4a4 · 0944f4a4
Commit 0944f4a4 authored Nov 10, 2014 by Masayuki Tanaka
Hide whitespace changes
Inline Side-by-side

Showing with 8 additions and 2 deletions

c3.js c3.js +4 -1

c3.min.js c3.min.js +0 -0

tooltip.js src/tooltip.js +4 -1

No files found.
--- a/c3.js
+++ b/c3.js
@@ -3589,10 +3589,13 @@
            if (tooltipRight > chartRight) {
                tooltipLeft -= tooltipRight - chartRight;
            }
-            if (tooltipTop + tHeight > $$.currentHeight && tooltipTop > tHeight + 30) {
+            if (tooltipTop + tHeight > $$.currentHeight) {
                tooltipTop -= tHeight + 30;
            }
        }
+        if (tooltipTop < 0) {
+            tooltipTop = 0;
+        }
        // Set tooltip
        $$.tooltip
            .style("top", tooltipTop + "px")

--- a/c3.min.js
+++ b/c3.min.js
--- a/src/tooltip.js
+++ b/src/tooltip.js
@@ -83,10 +83,13 @@ c3_chart_internal_fn.showTooltip = function (selectedData, mouse) {
        if (tooltipRight > chartRight) {
            tooltipLeft -= tooltipRight - chartRight;
        }
-        if (tooltipTop + tHeight > $$.currentHeight && tooltipTop > tHeight + 30) {
+        if (tooltipTop + tHeight > $$.currentHeight) {
            tooltipTop -= tHeight + 30;
        }
    }
+    if (tooltipTop < 0) {
+        tooltipTop = 0;
+    }
    // Set tooltip
    $$.tooltip
        .style("top", tooltipTop + "px")