Skip to content

Commit

Permalink
Do not forcibly hide controls in the dashboard
Browse files Browse the repository at this point in the history
Our views respect view=compact now entirely so there is no need for such
a generic "solution" since it also prevents some views from being
completely displayed in the dashboard such as the host, service and
contact detail view.

refs #7876
  • Loading branch information
Johannes Meyer committed Apr 20, 2015
1 parent 4b43fbe commit 5196f16
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
2 changes: 1 addition & 1 deletion public/css/icinga/layout-structure.less
Expand Up @@ -163,7 +163,7 @@ html {
}

.dashboard .controls {
display: none;
padding: 0;
}

/* Not growing larger than 3840px at 1em=16px right now */
Expand Down
8 changes: 5 additions & 3 deletions public/js/icinga/ui.js
Expand Up @@ -716,8 +716,6 @@
},

initializeControls: function (parent) {

var self = this;
if ($(parent).closest('.dashboard').length) {
return;
}
Expand Down Expand Up @@ -747,7 +745,6 @@
},

fixControls: function ($parent) {

var self = this;

if ('undefined' === typeof $parent) {
Expand All @@ -773,6 +770,11 @@

$('.controls', $parent).each(function (idx, el) {
var $el = $(el);

if ($el.closest('.dashboard').length) {
return;
}

var $fake = $el.next('.fake-controls');
var y = $parent.scrollTop();

Expand Down

0 comments on commit 5196f16

Please sign in to comment.