diff --git a/CHANGELOG.md b/CHANGELOG.md index 2ccae3cf6ae..f28fb65b492 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,3 +1,9 @@ +# 3.0.0-beta5 (unreleased) + +### Bug fixes +* **Graph**: Fixed issue with light theme text color issue in tooltip, fixes [#4702](https://github.com/grafana/grafana/issues/4702) +* **Snapshot**: Fixed issue with empty snapshots, fixes [#4706](https://github.com/grafana/grafana/issues/4706) + # 3.0.0-beta4 (2016-04-13) ### Bug fixes diff --git a/public/app/features/panel/metrics_panel_ctrl.ts b/public/app/features/panel/metrics_panel_ctrl.ts index df4e912211e..e8c16fd9ab7 100644 --- a/public/app/features/panel/metrics_panel_ctrl.ts +++ b/public/app/features/panel/metrics_panel_ctrl.ts @@ -65,7 +65,7 @@ class MetricsPanelCtrl extends PanelCtrl { var data = this.panel.snapshotData; // backward compatability if (!_.isArray(data)) { - data = data; + data = data.data; } this.events.emit('data-snapshot-load', data); diff --git a/public/app/plugins/panel/graph/module.ts b/public/app/plugins/panel/graph/module.ts index b626cc8d767..1fcd44204f7 100644 --- a/public/app/plugins/panel/graph/module.ts +++ b/public/app/plugins/panel/graph/module.ts @@ -161,7 +161,7 @@ class GraphCtrl extends MetricsPanelCtrl { onDataSnapshotLoad(snapshotData) { this.annotationsPromise = this.annotationsSrv.getAnnotations(this.dashboard); - this.onDataReceived(snapshotData.data); + this.onDataReceived(snapshotData); } onDataError(err) { diff --git a/public/app/plugins/panel/pluginlist/module.html b/public/app/plugins/panel/pluginlist/module.html index d9a52c29ac1..c73da35b391 100644 --- a/public/app/plugins/panel/pluginlist/module.html +++ b/public/app/plugins/panel/pluginlist/module.html @@ -1,30 +1,30 @@
-
-
- {{category.header}} -
-
-
- - - {{plugin.name}} - v{{plugin.info.version}} - - - Update available! - - - Enable now - - - Up to date - -
-
-
- - No additional panels installed. Browse Grafana.net - -
-
+
+
+ {{category.header}} +
+
+ + + + {{plugin.name}} + v{{plugin.info.version}} + + + Update available! + + + Enable now + + + Up to date + + +
+
+ + No additional panels installed. Browse Grafana.net + +
+
diff --git a/public/app/plugins/panel/pluginlist/module.ts b/public/app/plugins/panel/pluginlist/module.ts index 9ad43b25e56..8ec86efd9c2 100644 --- a/public/app/plugins/panel/pluginlist/module.ts +++ b/public/app/plugins/panel/pluginlist/module.ts @@ -35,12 +35,14 @@ class PluginListCtrl extends PanelCtrl { this.addEditorTab('Options', 'public/app/plugins/panel/pluginlist/editor.html'); } - gotoPlugin(plugin) { - this.$location.path(`plugins/${plugin.id}/edit`); + gotoPlugin(plugin, evt) { + if (evt) { evt.stopPropagation(); } + this.$location.url(`plugins/${plugin.id}/edit`); } updateAvailable(plugin, $event) { $event.stopPropagation(); + $event.preventDefault(); var modalScope = this.$scope.$new(true); modalScope.plugin = plugin; diff --git a/public/app/plugins/panel/singlestat/module.ts b/public/app/plugins/panel/singlestat/module.ts index cd85fda7c92..8af684c9d68 100644 --- a/public/app/plugins/panel/singlestat/module.ts +++ b/public/app/plugins/panel/singlestat/module.ts @@ -56,7 +56,7 @@ class SingleStatCtrl extends MetricsPanelCtrl { this.events.on('data-received', this.onDataReceived.bind(this)); this.events.on('data-error', this.onDataError.bind(this)); - this.events.on('data-snapshot-load', this.onDataSnapshotLoad.bind(this)); + this.events.on('data-snapshot-load', this.onDataReceived.bind(this)); this.events.on('init-edit-mode', this.onInitEditMode.bind(this)); } @@ -71,10 +71,6 @@ class SingleStatCtrl extends MetricsPanelCtrl { this.render(); } - onDataSnapshotLoad(snapshotData) { - this.onDataReceived(snapshotData.data); - } - onDataError(err) { this.onDataReceived({data: []}); } diff --git a/public/app/plugins/panel/table/module.ts b/public/app/plugins/panel/table/module.ts index d9bba0576b7..7a2973cc390 100644 --- a/public/app/plugins/panel/table/module.ts +++ b/public/app/plugins/panel/table/module.ts @@ -60,7 +60,7 @@ class TablePanelCtrl extends MetricsPanelCtrl { this.events.on('data-received', this.onDataReceived.bind(this)); this.events.on('data-error', this.onDataError.bind(this)); - this.events.on('data-snapshot-load', this.onDataSnapshotLoad.bind(this)); + this.events.on('data-snapshot-load', this.onDataReceived.bind(this)); this.events.on('init-edit-mode', this.onInitEditMode.bind(this)); this.events.on('init-panel-actions', this.onInitPanelActions.bind(this)); } @@ -86,10 +86,6 @@ class TablePanelCtrl extends MetricsPanelCtrl { return super.issueQueries(datasource); } - onDataSnapshotLoad(data) { - this.onDataReceived(data.data); - } - onDataError(err) { this.dataRaw = []; this.render(); diff --git a/public/sass/_variables.dark.scss b/public/sass/_variables.dark.scss index bb5fdc0b026..560f2d2dab1 100644 --- a/public/sass/_variables.dark.scss +++ b/public/sass/_variables.dark.scss @@ -102,7 +102,9 @@ $tight-form-func-bg: #333; $tight-form-func-highlight-bg: #444; $modal-background: $black; -$code-tag-bg: #444; +$code-tag-bg: $dark-5; +$code-tag-border: lighten($code-tag-bg, 2%); + // Lists $grafanaListBackground: $dark-3; diff --git a/public/sass/_variables.light.scss b/public/sass/_variables.light.scss index e2e374b48ee..a7b5c72af2b 100644 --- a/public/sass/_variables.light.scss +++ b/public/sass/_variables.light.scss @@ -109,7 +109,8 @@ $tight-form-func-bg: $gray-5; $tight-form-func-highlight-bg: $gray-6; $modal-background: $body-bg; -$code-tag-bg: $dark-5; +$code-tag-bg: $gray-6; +$code-tag-border: darken($code-tag-bg, 3%); // Lists $grafanaListBackground: $gray-6; @@ -281,7 +282,7 @@ $tooltipColor: $popover-help-color; $tooltipBackground: $popover-help-bg; $tooltipArrowWidth: 5px; $tooltipArrowColor: $tooltipBackground; -$tooltipLinkColor: $link-color; +$tooltipLinkColor: lighten($popover-help-color, 5%); $graph-tooltip-bg: $gray-5; // images diff --git a/public/sass/base/_code.scss b/public/sass/base/_code.scss index 464eed966ce..eb085c0577c 100644 --- a/public/sass/base/_code.scss +++ b/public/sass/base/_code.scss @@ -10,7 +10,7 @@ pre { font-size: $font-size-base - 2; background-color: $code-tag-bg; color: $text-color; - border: 1px solid darken($code-tag-bg, 15%); + border: 1px solid $code-tag-border; padding: 10px; border-radius: 4px; } diff --git a/public/sass/components/_panel_graph.scss b/public/sass/components/_panel_graph.scss index 12215f16798..ca79034a0bd 100644 --- a/public/sass/components/_panel_graph.scss +++ b/public/sass/components/_panel_graph.scss @@ -236,6 +236,7 @@ white-space: nowrap; font-size: $font-size-sm; background-color: $graph-tooltip-bg; + color: $text-color; .graph-tooltip-time { text-align: center; @@ -248,7 +249,7 @@ display: table-row; &--highlight { - color: $link-color; + color: $text-color-emphasis; font-weight: bold; } }