Skip to content

Commit

Permalink
fix(gridMenu): menu at wrong place when using Frozen Col 0, fixes #436
Browse files Browse the repository at this point in the history
  • Loading branch information
@ghiscoding
ghiscoding committed Sep 18, 2019
1 parent 106c79a commit 7bd6f27
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion controls/slick.gridmenu.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@
_gridOptions = grid.getOptions();
var gridMenuWidth = (_options.gridMenu && _options.gridMenu.menuWidth) || _defaults.menuWidth;
var $header;
if (_gridOptions && _gridOptions.frozenColumn && _gridOptions.frozenColumn > 0) {
if (_gridOptions && _gridOptions.frozenColumn && _gridOptions.frozenColumn >= 0) {
$header = $('.' + _gridUid + ' .slick-header-right');
} else {
$header = $('.' + _gridUid + ' .slick-header-left');
Expand Down

1 comment on commit 7bd6f27

@songjiqiang
Copy link

@songjiqiang songjiqiang commented on 7bd6f27 Sep 18, 2019 •
edited
Loading

Choose a reason for hiding this comment

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

If frozenColumn is set to 0, this judgment is problematic.For example, if(0) is actually false, then the judgment logic should be deleted, Please check the screenshot of my question, thank you
图片

Please sign in to comment.