From ff9721aefd053455e9a7fb863e949d53a7df8734 Mon Sep 17 00:00:00 2001
From: 1ambda <1amb4a@gmail.com>
Date: Sun, 21 May 2017 11:32:39 +0900
Subject: [PATCH 1/4] style: Reformat html

---
 .../src/components/rename/rename.controller.js    |  2 ++
 zeppelin-web/src/components/rename/rename.css     |  0
 zeppelin-web/src/components/rename/rename.html    | 15 ++++++++-------
 3 files changed, 10 insertions(+), 7 deletions(-)
 create mode 100644 zeppelin-web/src/components/rename/rename.css

diff --git a/zeppelin-web/src/components/rename/rename.controller.js b/zeppelin-web/src/components/rename/rename.controller.js
index 305c5198013..2682b6e4118 100644
--- a/zeppelin-web/src/components/rename/rename.controller.js
+++ b/zeppelin-web/src/components/rename/rename.controller.js
@@ -12,6 +12,8 @@
  * limitations under the License.
  */
 
+import './rename.css'
+
 angular.module('zeppelinWebApp').controller('RenameCtrl', RenameCtrl)
 
 function RenameCtrl ($scope) {
diff --git a/zeppelin-web/src/components/rename/rename.css b/zeppelin-web/src/components/rename/rename.css
new file mode 100644
index 00000000000..e69de29bb2d
diff --git a/zeppelin-web/src/components/rename/rename.html b/zeppelin-web/src/components/rename/rename.html
index 723c6aa9250..2de79fd4278 100644
--- a/zeppelin-web/src/components/rename/rename.html
+++ b/zeppelin-web/src/components/rename/rename.html
@@ -11,28 +11,29 @@
 See the License for the specific language governing permissions and
 limitations under the License.
 -->
-<div id="renameModal" class="modal fade" role="dialog"
-     tabindex="-1">
+<div id="renameModal" class="modal fade" role="dialog" tabindex="-1">
   <div class="modal-dialog">
     <div class="modal-content">
+      <!-- modal-header -->
       <div class="modal-header">
         <button type="button" class="close" data-dismiss="modal">&times;</button>
         <h4 class="modal-title">{{title}}</h4>
       </div>
+
+      <!-- modal-body -->
       <div class="modal-body">
         <label ng-if="isValid">Please enter a new name</label>
-          <label class="text-danger" ng-if="!isValid">Please enter a valid name</label>
+        <label class="text-danger" ng-if="!isValid">Please enter a valid name</label>
         <div class="form-group" ng-class="{'has-error': !isValid}">
           <input type="text" class="form-control"
-            ng-model="params.newName" ng-change="validate()"
-            ng-keyup="$event.keyCode == 13 && isValid && rename()" />
+                 ng-model="params.newName" ng-change="validate()"
+                 ng-keyup="$event.keyCode == 13 && isValid && rename()" />
         </div>
-
       </div>
       <div class="modal-footer">
         <div>
           <button type="button" class="btn btn-default btn-primary"
-            ng-click="rename()" ng-class="{'disabled': !isValid}">
+                  ng-click="rename()" ng-class="{'disabled': !isValid}">
             Rename
           </button>
         </div>

From d310174c11dd24745ff30e448136f373588df921 Mon Sep 17 00:00:00 2001
From: 1ambda <1amb4a@gmail.com>
Date: Sun, 21 May 2017 11:39:48 +0900
Subject: [PATCH 2/4] feat: Style rename dialog body

---
 zeppelin-web/src/components/rename/rename.css | 24 +++++++++++++++++++
 .../src/components/rename/rename.html         |  6 ++---
 2 files changed, 27 insertions(+), 3 deletions(-)

diff --git a/zeppelin-web/src/components/rename/rename.css b/zeppelin-web/src/components/rename/rename.css
index e69de29bb2d..6f8c73261d5 100644
--- a/zeppelin-web/src/components/rename/rename.css
+++ b/zeppelin-web/src/components/rename/rename.css
@@ -0,0 +1,24 @@
+.modal-header-rename {
+  background-color: #3071a9;
+  border: 2px solid #3071a9;
+  border-top-left-radius: 4px;
+  border-top-right-radius: 4px;
+}
+
+.modal-header-rename .close {
+  color: #cfcfcf;
+  opacity: 1;
+}
+
+.modal-header-rename .modal-title {
+  color: white;
+  font-size: 20px;
+  font-weight: 300;
+}
+
+.modal-body-rename label {
+  font-size: 17px;
+  font-weight: 400;
+  margin-top: 5px;
+  margin-bottom: 15px;
+}
diff --git a/zeppelin-web/src/components/rename/rename.html b/zeppelin-web/src/components/rename/rename.html
index 2de79fd4278..5fdfba94ee8 100644
--- a/zeppelin-web/src/components/rename/rename.html
+++ b/zeppelin-web/src/components/rename/rename.html
@@ -15,15 +15,15 @@
   <div class="modal-dialog">
     <div class="modal-content">
       <!-- modal-header -->
-      <div class="modal-header">
+      <div class="modal-header modal-header-rename">
         <button type="button" class="close" data-dismiss="modal">&times;</button>
         <h4 class="modal-title">{{title}}</h4>
       </div>
 
       <!-- modal-body -->
-      <div class="modal-body">
+      <div class="modal-body modal-body-rename">
         <label ng-if="isValid">Please enter a new name</label>
-        <label class="text-danger" ng-if="!isValid">Please enter a valid name</label>
+        <label ng-if="!isValid" class="text-danger">Please enter a valid name</label>
         <div class="form-group" ng-class="{'has-error': !isValid}">
           <input type="text" class="form-control"
                  ng-model="params.newName" ng-change="validate()"

From 0fa0caad806b39f514ebcd5f63a83a03c663d7e9 Mon Sep 17 00:00:00 2001
From: 1ambda <1amb4a@gmail.com>
Date: Sun, 21 May 2017 16:43:01 +0900
Subject: [PATCH 3/4] fix: RAT issue

---
 zeppelin-web/src/components/rename/rename.css | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/zeppelin-web/src/components/rename/rename.css b/zeppelin-web/src/components/rename/rename.css
index 6f8c73261d5..45d4710164e 100644
--- a/zeppelin-web/src/components/rename/rename.css
+++ b/zeppelin-web/src/components/rename/rename.css
@@ -1,3 +1,17 @@
+/*
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 .modal-header-rename {
   background-color: #3071a9;
   border: 2px solid #3071a9;

From 0e863d7870ab76568941feb7bd9e2c2d803ecfab Mon Sep 17 00:00:00 2001
From: 1ambda <1amb4a@gmail.com>
Date: Mon, 22 May 2017 18:11:42 +0900
Subject: [PATCH 4/4] fix: Use ng-enter

---
 zeppelin-web/src/components/rename/rename.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/zeppelin-web/src/components/rename/rename.html b/zeppelin-web/src/components/rename/rename.html
index 5fdfba94ee8..71caa761be9 100644
--- a/zeppelin-web/src/components/rename/rename.html
+++ b/zeppelin-web/src/components/rename/rename.html
@@ -27,7 +27,7 @@ <h4 class="modal-title">{{title}}</h4>
         <div class="form-group" ng-class="{'has-error': !isValid}">
           <input type="text" class="form-control"
                  ng-model="params.newName" ng-change="validate()"
-                 ng-keyup="$event.keyCode == 13 && isValid && rename()" />
+                 ng-enter="isValid && rename()" />
         </div>
       </div>
       <div class="modal-footer">