From 5f0a461a3ebf81a36cb126cf216f3ed9a8231f9f Mon Sep 17 00:00:00 2001
From: astroshim <hsshim@nflabs.com>
Date: Sat, 17 Sep 2016 00:53:21 +0900
Subject: [PATCH 1/3] fix align checkboxs.

---
 .../interpreter-create.html                   | 76 ++++++++++---------
 1 file changed, 42 insertions(+), 34 deletions(-)

diff --git a/zeppelin-web/src/app/interpreter/interpreter-create/interpreter-create.html b/zeppelin-web/src/app/interpreter/interpreter-create/interpreter-create.html
index 1550126b9b7..a4fe38b34c6 100644
--- a/zeppelin-web/src/app/interpreter/interpreter-create/interpreter-create.html
+++ b/zeppelin-web/src/app/interpreter/interpreter-create/interpreter-create.html
@@ -68,55 +68,63 @@ <h4>Create new interpreter</h4>
           </span>
           <span>Interpreter for note</span>
         </div>
-        <br />
 
-        <div class="col-md-12" style="padding-left:0px">
-          <div class="checkbox">
-            <span class="input-group" style="line-height:30px;">
-              <label><input type="checkbox" style="width:20px" ng-model="newInterpreterSetting.option.isExistingProcess"/> Connect to existing process </label>
-            </span>
+        <div class="row interpreter" style="margin-top: 5px;">
+          <div class="col-md-12">
+            <div class="checkbox remove-margin-top-bottom">
+          <span class="input-group" style="line-height:30px;">
+            <label>
+              <input type="checkbox" style="width:20px" id="isExistingProcess" ng-model="newInterpreterSetting.option.isExistingProcess"/>
+              Connect to existing process
+            </label>
+          </span>
+            </div>
           </div>
         </div>
-        <div ng-show="newInterpreterSetting.option.isExistingProcess" class="form-group" style="width:200px">
-          <b>Host</b>
+        <div class="row interpreter" ng-show="newInterpreterSetting.option.isExistingProcess" >
+          <div class="col-md-12">
+            <b>Host</b>
             <input id="newInterpreterSettingHost" input pu-elastic-input
-              pu-elastic-input-minwidth="180px" ng-model="newInterpreterSetting.option.host" />
-        </div>
-        <div ng-show="newInterpreterSetting.option.isExistingProcess" class="form-group" style="width:200px">
-          <b>Port</b>
-           <input id="newInterpreterSettingPort" input pu-elastic-input
-            pu-elastic-input-minwidth="180px" ng-model="newInterpreterSetting.option.port" />
+                   pu-elastic-input-minwidth="180px" ng-model="newInterpreterSetting.option.host"/>
+          </div>
+          <div class="col-md-12">
+            <b>Port</b>
+            <input id="newInterpreterSettingPort" input pu-elastic-input
+                   pu-elastic-input-minwidth="180px" ng-model="newInterpreterSetting.option.port"/>
+          </div>
         </div>
-
-        <div class="col-md-12">
-          <div class="checkbox">
+        <div class="row interpreter">
+          <div class="col-md-12">
+            <div class="checkbox remove-margin-top-bottom">
           <span class="input-group" style="line-height:30px;">
-            <label><input type="checkbox" style="width:18px !important" id="idShowPermission" ng-click="togglePermissions('newInterpreter')" ng-model="newInterpreterSetting.option.setPermission"/>
-            Set permission </label>
+            <label>
+              <input type="checkbox" style="width:20px !important" id="idShowPermission" ng-click="togglePermissions('newInterpreter')" ng-model="newInterpreterSetting.option.setPermission"/>
+               Set permission
+            </label>
           </span>
+            </div>
           </div>
         </div>
-
-        <div class="col-md-12">
-          <!-- permissions -->
-          <div ng-show="newInterpreterSetting.option.setPermission" class="permissionsForm">
-            <div>
-              <p>
-                Enter comma separated users in the fields. <br />
-                Empty field (*) implies anyone can run this interpreter.
-              </p>
+        <div class="row interpreter">
+          <div class="col-md-12">
+            <!-- permissions -->
+            <div ng-show="newInterpreterSetting.option.setPermission" class="permissionsForm">
               <div>
-
-                <span class="owners">Owners </span>
-                <select id="newInterpreterUsers"  class="form-control" multiple="multiple">
-                  <option ng-repeat="user in newInterpreterSetting.option.users" selected="selected">{{user}}</option>
-                </select>
+                <p>
+                  Enter comma separated users in the fields. <br />
+                  Empty field (*) implies anyone can run this interpreter.
+                </p>
+                <div>
+                  <span class="owners">Owners </span>
+                  <select id="newInterpreterUsers" class="form-control" multiple="multiple">
+                    <option ng-repeat="user in newInterpreterSetting.option.users" selected="selected">{{user}}</option>
+                  </select>
+                </div>
               </div>
             </div>
           </div>
         </div>
 
-
         <b>Properties</b>
         <table class="table table-striped properties">
           <tr>

From 2d3ec76702c1127072f92489f5be333f7d85679f Mon Sep 17 00:00:00 2001
From: astroshim <hsshim@nflabs.com>
Date: Sun, 18 Sep 2016 22:58:21 +0900
Subject: [PATCH 2/3] fix space

---
 .../app/interpreter/interpreter-create/interpreter-create.html  | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/zeppelin-web/src/app/interpreter/interpreter-create/interpreter-create.html b/zeppelin-web/src/app/interpreter/interpreter-create/interpreter-create.html
index a4fe38b34c6..11763422d48 100644
--- a/zeppelin-web/src/app/interpreter/interpreter-create/interpreter-create.html
+++ b/zeppelin-web/src/app/interpreter/interpreter-create/interpreter-create.html
@@ -105,6 +105,8 @@ <h4>Create new interpreter</h4>
             </div>
           </div>
         </div>
+        <br/>
+
         <div class="row interpreter">
           <div class="col-md-12">
             <!-- permissions -->

From fada36bc7076c040bbc903fbb1708d5b448204d2 Mon Sep 17 00:00:00 2001
From: astroshim <hsshim@nflabs.com>
Date: Mon, 19 Sep 2016 16:42:47 +0900
Subject: [PATCH 3/3] fix spaces

---
 .../interpreter-create.html                   | 148 +++++++++---------
 1 file changed, 76 insertions(+), 72 deletions(-)

diff --git a/zeppelin-web/src/app/interpreter/interpreter-create/interpreter-create.html b/zeppelin-web/src/app/interpreter/interpreter-create/interpreter-create.html
index 11763422d48..d0e0749360f 100644
--- a/zeppelin-web/src/app/interpreter/interpreter-create/interpreter-create.html
+++ b/zeppelin-web/src/app/interpreter/interpreter-create/interpreter-create.html
@@ -35,8 +35,8 @@ <h4>Create new interpreter</h4>
           </select>
         </div>
 
-        <b>Option</b>
         <div>
+          <h5>Option</h5>
           <span class="btn-group">
             <button type="button" class="btn btn-default btn-xs dropdown-toggle"
                     data-toggle="dropdown">
@@ -127,81 +127,85 @@ <h4>Create new interpreter</h4>
           </div>
         </div>
 
-        <b>Properties</b>
-        <table class="table table-striped properties">
-          <tr>
-            <th>name</th>
-            <th>value</th>
-            <th>description</th>
-            <th>action</th>
-          </tr>
-          <tr ng-repeat="(key, value) in newInterpreterSetting.properties">
-            <td>{{key}}</td>
-            <td><textarea msd-elastic ng-model="value.value"></textarea></td>
-            <td>{{value.description}}</td>
-            <td>
-              <button class="btn btn-default btn-sm fa fa-remove" ng-click="removeInterpreterProperty(key)">
-              </button>
-            </td>
-          </tr>
+        <div>
+          <h5>Properties</h5>
+          <table class="table table-striped properties">
+            <tr>
+              <th>name</th>
+              <th>value</th>
+              <th>description</th>
+              <th>action</th>
+            </tr>
+            <tr ng-repeat="(key, value) in newInterpreterSetting.properties">
+              <td>{{key}}</td>
+              <td><textarea msd-elastic ng-model="value.value"></textarea></td>
+              <td>{{value.description}}</td>
+              <td>
+                <button class="btn btn-default btn-sm fa fa-remove" ng-click="removeInterpreterProperty(key)">
+                </button>
+              </td>
+            </tr>
 
-          <tr>
-            <td>
-              <input pu-elastic-input pu-elastic-input-minwidth="180px"
-                     ng-model="newInterpreterSetting.propertyKey" />
-            </td>
-            <td><textarea msd-elastic ng-model="newInterpreterSetting.propertyValue"></textarea></td>
-            <td></td>
-            <td>
-              <button class="btn btn-default btn-sm fa fa-plus" ng-click="addNewInterpreterProperty()">
-              </button>
-            </td>
-          </tr>
-        </table>
+            <tr>
+              <td>
+                <input pu-elastic-input pu-elastic-input-minwidth="180px"
+                       ng-model="newInterpreterSetting.propertyKey" />
+              </td>
+              <td><textarea msd-elastic ng-model="newInterpreterSetting.propertyValue"></textarea></td>
+              <td></td>
+              <td>
+                <button class="btn btn-default btn-sm fa fa-plus" ng-click="addNewInterpreterProperty()">
+                </button>
+              </td>
+            </tr>
+          </table>
+        </div>
 
-        <b>Dependencies</b>
-        <table class="table table-striped properties">
-          <tr>
-            <th>artifact</th>
-            <th>exclude</th>
-            <th>action</th>
-          </tr>
+        <div>
+          <h5>Dependencies</h5>
+          <table class="table table-striped properties">
+            <tr>
+              <th>artifact</th>
+              <th>exclude</th>
+              <th>action</th>
+            </tr>
 
-          <tr ng-repeat="dep in newInterpreterSetting.dependencies">
-            <td>
-              <input ng-model="dep.groupArtifactVersion" style="width:100%" />
-            </td>
-            <td>
-              <textarea msd-elastic ng-model="dep.exclusions"
-                        ng-list
-                        placeholder="(Optional) comma separated groupId:artifactId list">
-              </textarea>
-            </td>
-            <td>
-              <button class="btn btn-default btn-sm fa fa-remove"
-                   ng-click="removeInterpreterDependency(dep.groupArtifactVersion)">
-              </button>
-            </td>
-          </tr>
+            <tr ng-repeat="dep in newInterpreterSetting.dependencies">
+              <td>
+                <input ng-model="dep.groupArtifactVersion" style="width:100%" />
+              </td>
+              <td>
+                <textarea msd-elastic ng-model="dep.exclusions"
+                          ng-list
+                          placeholder="(Optional) comma separated groupId:artifactId list">
+                </textarea>
+              </td>
+              <td>
+                <button class="btn btn-default btn-sm fa fa-remove"
+                     ng-click="removeInterpreterDependency(dep.groupArtifactVersion)">
+                </button>
+              </td>
+            </tr>
 
-          <tr>
-            <td>
-              <input ng-model="newInterpreterSetting.depArtifact"
-                     placeholder="groupId:artifactId:version or local file path"
-                     style="width: 100%" />
-            </td>
-            <td>
-              <textarea msd-elastic ng-model="newInterpreterSetting.depExclude"
-                        ng-list
-                        placeholder="(Optional) comma separated groupId:artifactId list">
-              </textarea>
-            </td>
-            <td>
-              <button class="btn btn-default btn-sm fa fa-plus" ng-click="addNewInterpreterDependency()">
-              </button>
-            </td>
-          </tr>
-        </table>
+            <tr>
+              <td>
+                <input ng-model="newInterpreterSetting.depArtifact"
+                       placeholder="groupId:artifactId:version or local file path"
+                       style="width: 100%" />
+              </td>
+              <td>
+                <textarea msd-elastic ng-model="newInterpreterSetting.depExclude"
+                          ng-list
+                          placeholder="(Optional) comma separated groupId:artifactId list">
+                </textarea>
+              </td>
+              <td>
+                <button class="btn btn-default btn-sm fa fa-plus" ng-click="addNewInterpreterDependency()">
+                </button>
+              </td>
+            </tr>
+          </table>
+        </div>
 
         <span class="btn btn-primary" ng-click="addNewInterpreterSetting()">
           Save