Skip to content

Commit

Permalink
fix dojo#886 ColumnHider on OSX disappeared, with latest css changes
Browse files Browse the repository at this point in the history
We were adding the dgrid-header class to the headerScrollNode, which
will hide the ColumnHider icon on OSX browsers that hide the scrollbars
by default, because we set overflow:hidden in dgrid-header.

This is regression that appeared in v0.3.14, it was not present in
v0.3.13.
  • Loading branch information
James Morrin committed Apr 4, 2014
1 parent 0ff0a13 commit a11a156
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion List.js
Original file line number Diff line number Diff line change
Expand Up @@ -256,7 +256,7 @@ function(kernel, declare, dom, listen, has, miscUtil, TouchScroll, hasClass, put
bodyNode.tabIndex = -1;
}

this.headerScrollNode = put(domNode, "div.dgrid-header.dgrid-header-scroll.dgrid-scrollbar-width" +
this.headerScrollNode = put(domNode, "div.dgrid-header-scroll.dgrid-scrollbar-width" +
(addUiClasses ? ".ui-widget-header" : ""));

// Place footer node (initially hidden if showFooter is false).
Expand Down

0 comments on commit a11a156

Please sign in to comment.