From 36a6714e9e5a57480cf9bf86a9fa4ccd65ca1c54 Mon Sep 17 00:00:00 2001
From: Alanna Scott <alanna.scott@airbnb.com>
Date: Thu, 25 Aug 2016 11:22:11 -0700
Subject: [PATCH] slice/explore header (#996)

* move slice header to partial, show datasource in slice title area if no slice is saved

* change partial name, and use same styling as slice title

* use jinja style guide spacing rule

* use [datasource] - untitled
---
 caravel/templates/caravel/explore.html        | 31 +------------------
 .../caravel/partials/_explore_title.html      | 30 ++++++++++++++++++
 2 files changed, 31 insertions(+), 30 deletions(-)
 create mode 100644 caravel/templates/caravel/partials/_explore_title.html

diff --git a/caravel/templates/caravel/explore.html b/caravel/templates/caravel/explore.html
index 52da875b4eb09..ec1010c45b10d 100644
--- a/caravel/templates/caravel/explore.html
+++ b/caravel/templates/caravel/explore.html
@@ -188,36 +188,7 @@
             <div class="panel-title">
               <div class="row">
                 <div class="col-md-6">
-                  {% if slice %}
-                    {% if slice.slice_name %}
-                      <h2>
-                        {{ slice.slice_name }}
-                        <small class="star-edit-icons">
-                          <span class="favstar" obj_id="{{ slice.id }}"></span>
-                          <span>
-                            <a
-                              href="/slicemodelview/edit/{{ slice.id }}"
-                              data-toggle="tooltip"
-                              title="Edit Description"
-                              class="edit-slice-description-icon"
-                            >
-                              {% if slice.description %}
-                                <i
-                                  class="fa fa-question-circle-o"
-                                  data-toggle="tooltip"
-                                  data-placement="bottom"
-                                  title="{{ slice.description }}">
-                                </i>
-                              {% endif %}
-                              <i class="fa fa-edit"></i>
-                            </a>
-                          </span>
-                        </small>
-                      </h2>
-                    {% else %}
-                      <h2>untitled</h2>
-                    {% endif %}
-                  {% endif %}
+                  {% include 'caravel/partials/_explore_title.html' %}
                 </div>
                 <div class="col-md-6">
                   <div class="slice-meta-controls pull-right">
diff --git a/caravel/templates/caravel/partials/_explore_title.html b/caravel/templates/caravel/partials/_explore_title.html
new file mode 100644
index 0000000000000..9f9767ce47e4c
--- /dev/null
+++ b/caravel/templates/caravel/partials/_explore_title.html
@@ -0,0 +1,30 @@
+{% if slice %}
+  {% if slice.slice_name %}
+    <h2>
+      {{ slice.slice_name }}
+      <small class="star-edit-icons">
+        <span class="favstar" obj_id="{{ slice.id }}"></span>
+        <span>
+          <a
+            href="/slicemodelview/edit/{{ slice.id }}"
+            data-toggle="tooltip"
+            title="Edit Description"
+            class="edit-slice-description-icon"
+          >
+            {% if slice.description %}
+              <i
+                class="fa fa-question-circle-o"
+                data-toggle="tooltip"
+                data-placement="bottom"
+                title="{{ slice.description }}">
+              </i>
+            {% endif %}
+            <i class="fa fa-edit"></i>
+          </a>
+        </span>
+      </small>
+    </h2>
+  {% endif %}
+{% else %}
+  <h2>[{{ viz.datasource.table_name }}] - untitled</h2>
+{% endif %}