Skip to content

Commit

Permalink
Fix tooltip error with draggable columns
Browse files Browse the repository at this point in the history
- added draggable columns option to autotooltips example
- patched autotooltips plugin as per mleibman/SlickGrid#1056
- the HTML encoding mentioned in the pull request has been removed already in the current master
- thanks to B1nke
  • Loading branch information
6pac committed Apr 1, 2015
1 parent 0145d1a commit 9511a0f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion examples/example-autotooltips.html
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ <h2>View Source:</h2>

<script src="../lib/jquery-1.11.2.min.js"></script>
<script src="../lib/jquery.event.drag-2.2.js"></script>
<script src="../lib/jquery-ui-1.11.3.min.js"></script>

<script src="../slick.core.js"></script>
<script src="../slick.grid.js"></script>
Expand All @@ -53,7 +54,7 @@ <h2>View Source:</h2>

var options = {
enableCellNavigation: true,
enableColumnReorder: false
enableColumnReorder: true
};

$(function () {
Expand Down
2 changes: 1 addition & 1 deletion plugins/slick.autotooltips.js
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@
function handleHeaderMouseEnter(e, args) {
var column = args.column,
$node = $(e.target).closest(".slick-header-column");
if (!column.toolTip) {
if (column && !column.toolTip) {
$node.attr("title", ($node.innerWidth() < $node[0].scrollWidth) ? column.name : "");
}
}
Expand Down

0 comments on commit 9511a0f

Please sign in to comment.