From e2ad374e5bd306c4fc4a76660e3ca295ed8e01e0 Mon Sep 17 00:00:00 2001
From: Guo Xiang Tan <tgx_world@hotmail.com>
Date: Fri, 29 Mar 2019 11:54:49 +0800
Subject: [PATCH] UX: USe bar chart for admin search logs details page.

A bar chart is better when comparing the values across the X axis.
---
 .../javascripts/admin/components/admin-graph.js.es6       | 8 ++++++--
 .../javascripts/admin/templates/search-logs-term.hbs      | 2 +-
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/app/assets/javascripts/admin/components/admin-graph.js.es6 b/app/assets/javascripts/admin/components/admin-graph.js.es6
index 7752230cfb148..5fbcade709a8e 100644
--- a/app/assets/javascripts/admin/components/admin-graph.js.es6
+++ b/app/assets/javascripts/admin/components/admin-graph.js.es6
@@ -3,6 +3,8 @@ import { number } from "discourse/lib/formatter";
 
 export default Ember.Component.extend({
   tagName: "canvas",
+  type: "line",
+
   refreshChart() {
     const ctx = this.$()[0].getContext("2d");
     const model = this.get("model");
@@ -14,14 +16,16 @@ export default Ember.Component.extend({
         {
           data: rawData.map(r => r.y),
           label: model.get("title"),
-          backgroundColor: "rgba(200,220,240,0.3)",
+          backgroundColor: `rgba(200,220,240,${
+            this.get("type") === "bar" ? 1 : 0.3
+          })`,
           borderColor: "#08C"
         }
       ]
     };
 
     const config = {
-      type: "line",
+      type: this.get("type"),
       data: data,
       options: {
         responsive: true,
diff --git a/app/assets/javascripts/admin/templates/search-logs-term.hbs b/app/assets/javascripts/admin/templates/search-logs-term.hbs
index 5e640039c3442..8b9a6ccb94c41 100644
--- a/app/assets/javascripts/admin/templates/search-logs-term.hbs
+++ b/app/assets/javascripts/admin/templates/search-logs-term.hbs
@@ -9,7 +9,7 @@
 </h2>
 
 {{#conditional-loading-spinner condition=refreshing}}
-  {{admin-graph model=model}}
+  {{admin-graph model=model type="bar"}}
 
   <br><br>
   <h2> {{i18n "admin.logs.search_logs.header_search_results"}} </h2>