scrollbar: styles cleanup

This commit is contained in:
Alexander Zobnin 2018-04-06 20:00:03 +03:00
parent 9f07ae72ee
commit d3d64337b4
5 changed files with 10 additions and 12 deletions

View File

@ -77,7 +77,6 @@ export function geminiScrollbar() {
}); });
scope.$on('$destroy', () => { scope.$on('$destroy', () => {
// scrollbar.destroy();
scrollbar.dispose(); scrollbar.dispose();
}); });
}, },

View File

@ -125,6 +125,8 @@ module.directive('grafanaPanel', function($rootScope, $document, $timeout) {
barOnCls: '_scrollbar', barOnCls: '_scrollbar',
scrollingCls: '_scrolling', scrollingCls: '_scrolling',
}); });
panelScrollbar.scroll();
} }
}); });

View File

@ -18,9 +18,7 @@ module.directive('graphLegend', function(popoverSrv, $timeout) {
const legendRightDefaultWidth = 10; const legendRightDefaultWidth = 10;
scope.$on('$destroy', function() { scope.$on('$destroy', function() {
if (legendScrollbar) { destroyScrollbar();
legendScrollbar.dispose();
}
}); });
ctrl.events.on('render-legend', () => { ctrl.events.on('render-legend', () => {
@ -288,6 +286,7 @@ module.directive('graphLegend', function(popoverSrv, $timeout) {
destroyScrollbar(); destroyScrollbar();
legendScrollbar = baron(scrollbarParams); legendScrollbar = baron(scrollbarParams);
} }
legendScrollbar.scroll();
} }
function destroyScrollbar() { function destroyScrollbar() {

View File

@ -58,7 +58,7 @@
position: relative; position: relative;
// fix for Firefox (white stripe on the right of scrollbar) // fix for Firefox (white stripe on the right of scrollbar)
width: 99%; width: calc(100% - 1px);
.popover-content { .popover-content {
padding: 0; padding: 0;
@ -67,6 +67,9 @@
.graph-legend-content { .graph-legend-content {
position: relative; position: relative;
// fix for Firefox (white stripe on the right of scrollbar)
width: calc(100% - 1px);
} }
.graph-legend-scroll { .graph-legend-scroll {

View File

@ -189,10 +189,6 @@
.baron { .baron {
display: inline-block; display: inline-block;
overflow: hidden; overflow: hidden;
// Width needs to be set to prevent content width issues
// Set to 99% instead of 100% for fixing Firefox issue (white stripe on the right of scrollbar)
width: 99%;
} }
// Fix for side menu on mobile devices // Fix for side menu on mobile devices
@ -203,7 +199,6 @@
.baron__clipper { .baron__clipper {
position: relative; position: relative;
overflow: hidden; overflow: hidden;
height: 100%;
} }
.baron__scroller { .baron__scroller {
@ -289,8 +284,8 @@
.baron.panel-content--scrollable { .baron.panel-content--scrollable {
// Width needs to be set to prevent content width issues // Width needs to be set to prevent content width issues
// Set to 99% instead of 100% for fixing Firefox issue (white stripe on the right of scrollbar) // Set to less than 100% for fixing Firefox issue (white stripe on the right of scrollbar)
width: 99%; width: calc(100% - 2px);
.baron__scroller { .baron__scroller {
padding-top: 1px; padding-top: 1px;