From 5bf8a36a8785332dc3287badab4050d028eb08af Mon Sep 17 00:00:00 2001 From: Whymarrh Whitby Date: Fri, 8 Mar 2019 15:04:19 -0330 Subject: [PATCH] Centre all notification popups --- app/scripts/lib/notification-manager.js | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/app/scripts/lib/notification-manager.js b/app/scripts/lib/notification-manager.js index 969a9459a135..a97e89ffc083 100644 --- a/app/scripts/lib/notification-manager.js +++ b/app/scripts/lib/notification-manager.js @@ -1,7 +1,6 @@ const extension = require('extensionizer') -const height = 620 -const width = 360 - +const NOTIFICATION_HEIGHT = 620 +const NOTIFICATION_WIDTH = 360 class NotificationManager { @@ -26,13 +25,19 @@ class NotificationManager { // bring focus to existing chrome popup extension.windows.update(popup.id, { focused: true }) } else { + const screenWidth = window.screen.width + const screenHeight = window.screen.height + const notificationTop = (screenHeight / 2) - (NOTIFICATION_HEIGHT / 2) + const notificationLeft = (screenWidth / 2) - (NOTIFICATION_WIDTH / 2) const cb = (currentPopup) => { this._popupId = currentPopup.id } // create new notification popup const creation = extension.windows.create({ url: 'notification.html', type: 'popup', - width, - height, + width: NOTIFICATION_WIDTH, + height: NOTIFICATION_HEIGHT, + top: Math.max(notificationTop, 0), + left: Math.max(notificationLeft, 0), }, cb) creation && creation.then && creation.then(cb) }