custom notifications: use text as property for action button label

tsullivan · tsullivan · commit 3c42f2d00ec5 · 2016-07-27T12:08:49.000-07:00
diff --git a/src/ui/public/notify/__tests__/notifier.js b/src/ui/public/notify/__tests__/notifier.js
@@ -16,10 +16,10 @@ describe('Notifier', function () {
     markdown: 'fooMardown',
     lifetime: 10000,
     customActions:[{
-      key: 'Cancel',
+      text: 'Cancel',
       callback: sinon.spy()
     }, {
-      key: 'OK',
+      text: 'OK',
       callback: sinon.spy()
     }]
   };
diff --git a/src/ui/public/notify/notifier.js b/src/ui/public/notify/notifier.js
@@ -96,8 +96,10 @@ function add(notif, cb) {
   } else if (notif.customActions) {
     // wrap all of the custom functions in a close
     notif.customActions = notif.customActions.map(action => {
-      action.callback = closeNotif(notif, action.callback, action.key);
-      return action;
+      return {
+        key: action.text,
+        callback: closeNotif(notif, action.callback, action.text)
+      };
     });
   }
 
@@ -383,10 +385,10 @@ Notifier.prototype.banner = function (msg, cb) {
  *   markdown: 'Some markdown content',
  *   type: 'info',
  *   customActions: [{
- *     key: 'next',
+ *     text: 'next',
  *     callback: function() { next(); }
  *   }, {
- *     key: 'prev',
+ *     text: 'prev',
  *     callback: function() { prev(); }
  *   }]
  * }
