address code review comments

src/core_plugins/kibana/public/dashboard/partials/save_dashboard.html

@@ -2,7 +2,7 @@
   role="form"
   ng-submit="opts.save()"
 >
-  <div class="localDropdownTitle">Save Dashboard</div>
+  <div class="localDropdownTitle">Save {{opts.dashboard.getDisplayName()}}</div>
   <input
     class="localDropdownInput"
     id="dashboardTitle"
@@ -18,7 +18,7 @@
     <label>
       <input type="checkbox" ng-model="opts.dashboard.timeRestore" ng-checked="opts.dashboard.timeRestore">
       Store time with {{opts.dashboard.getDisplayName()}}
-      <kbn-info info="Change the time filter to the currently selected time each time this dashboard is loaded"></kbn-info>
+      <kbn-info placement="bottom" info="Change the time filter to the currently selected time each time this dashboard is loaded"></kbn-info>
     </label>
   </div>
 

src/ui/public/courier/saved_object/saved_object.js

@@ -41,7 +41,7 @@ export default function SavedObjectFactory(es, kbnIndex, Promise, Private, Notif
     const type = config.type;
 
     self.getDisplayName = function () {
-      return type.charAt(0).toUpperCase() + type.slice(1);
+      return type;
     };
 
     // Create a notifier for sending alerts
@@ -239,16 +239,6 @@ export default function SavedObjectFactory(es, kbnIndex, Promise, Private, Notif
       return body;
     };
 
-    /**
-     * Checks the id of the instance. If it is not set, creates a uuid for it.
-     */
-    self.ensureId = function () {
-      this.id = this.id || uuid.v1();
-
-      // ensure that the docSource has the current id
-      docSource.id(self.id);
-    };
-
     /**
      * Returns true if the object's original title has been changed. New objects return false.
      * @return {boolean}
@@ -273,7 +263,11 @@ export default function SavedObjectFactory(es, kbnIndex, Promise, Private, Notif
       if (this.copyOnSave) {
         self.id = null;
       }
-      self.ensureId();
+
+      // Create a unique id for this object if it doesn't have one already.
+      this.id = this.id || uuid.v1();
+      // ensure that the docSource has the current id
+      docSource.id(self.id);
 
       let source = self.serialize();
 

src/ui/public/courier/saved_object/ui/saved_object_save_as_checkbox.html

@@ -0,0 +1,9 @@
+<div ng-hide="!savedObject.id">
+  <div ng-hide="!savedObject.isTitleChanged() || savedObject.copyOnSave" class="localDropdownWarning">
+    Note, saving a {{savedObject.getDisplayName()}} with a new name will no longer automatically create a copy.
+  </div>
+  <label>
+    <input type="checkbox" ng-model="savedObject.copyOnSave" ng-checked="savedObject.copyOnSave">
+    Save as a new {{savedObject.getDisplayName()}}
+  </label>
+</div>

src/ui/public/courier/saved_object/ui/saved_object_save_as_checkbox.js

@@ -1,20 +1,12 @@
 import uiModules from 'ui/modules';
+import saveObjectSaveAsCheckboxTemplate from './saved_object_save_as_checkbox.html';
 
 uiModules
   .get('kibana')
   .directive('savedObjectSaveAsCheckBox', function () {
     return {
       restrict: 'E',
-      template: `
-        <div class="form-group" ng-hide="!savedObject.id">
-          <div ng-hide="!savedObject.isTitleChanged() || savedObject.copyOnSave" class="localDropdownWarning">
-            Note, saving a {{savedObject.getDisplayName()}} with a new name will no longer automatically create a copy.
-          </div>
-          <label>
-            <input type="checkbox" ng-model="savedObject.copyOnSave" ng-checked="savedObject.copyOnSave">
-            Save as a new {{savedObject.getDisplayName()}}
-          </label>
-        </div>`,
+      template: saveObjectSaveAsCheckboxTemplate,
       scope: {
         savedObject: '='
       }