knight/datahub
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[#116,view/grid][s]: do not show extra header col if no scrollbar on grid body.

Browse Source
This commit is contained in:
Rufus Pollock 2012-05-17 23:57:08 +01:00
parent d49956e4de
commit fa0f37b5f7
2 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
css/grid.css
View File

@ -94,7 +94,7 @@ html>body thead.fixed-header tr {
/* set TBODY element to have block level attributes. All other non-IE browsers */
/* this enables overflow to work on TBODY element. All other non-IE, non-Mozilla browsers */
/* induced side effect is that child TDs no longer accept width: auto */
tbody.scrollContent {
tbody.scroll-content {
display: block;
max-height: 500px;
overflow: auto;

7
src/view-grid.js
View File

@ -167,7 +167,7 @@ my.Grid = Backbone.View.extend({
<th class="last-header" style="width: {{lastHeaderWidth}}px; padding: 0; margin: 0;"></th> \
</tr> \
</thead> \
<tbody class="scrollContent"></tbody> \
<tbody class="scroll-content"></tbody> \
</table> \
</div> \
',
@ -215,6 +215,11 @@ my.Grid = Backbone.View.extend({
});
newView.render();
});
// hide extra header col if no scrollbar to avoid unsightly overhang
var $tbody = this.el.find('tbody')[0];
if ($tbody.scrollHeight <= $tbody.offsetHeight) {
this.el.find('th.last-header').hide();
}
this.el.find('.recline-grid').toggleClass('no-hidden', (self.state.get('hiddenFields').length === 0));
return this;
},