time picker should be displayed only if index pattern has time field (#…

…13640) (#13749) * time picker should be displayed only if index pattern has time field * fixing based on review from lukas
elastic · Aug 29, 2017 · 826ca44 · 826ca44
1 parent 4a2d318
commit 826ca44
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/core_plugins/kibana/public/visualize/editor/editor.js b/src/core_plugins/kibana/public/visualize/editor/editor.js
@@ -209,10 +209,10 @@ function VisEditor($scope, $route, timefilter, AppState, $window, kbnUrl, courie
     };
 
     $scope.$watchMulti([
-      'searchSource.get("index").timeFieldName',
+      'searchSource.get("index")',
       'vis.type.options.showTimePicker',
-    ], function ([timeField, requiresTimePicker]) {
-      timefilter.enabled = Boolean(timeField || requiresTimePicker);
+    ], function ([index, requiresTimePicker]) {
+      timefilter.enabled = Boolean((!index || index.timeFieldName) && requiresTimePicker);
     });
 
     // update the searchSource when filters update

diff --git a/src/core_plugins/markdown_vis/public/markdown_vis.js b/src/core_plugins/markdown_vis/public/markdown_vis.js
@@ -34,6 +34,9 @@ function MarkdownVisProvider(Private) {
     editorConfig: {
       optionsTemplate: markdownVisParamsTemplate
     },
+    options: {
+      showTimePicker: false,
+    },
     requestHandler: 'none',
     implementsRenderComplete: true,
   });