Skip to content

Commit

Permalink
Centre all notification popups
Browse files Browse the repository at this point in the history
  • Loading branch information
whymarrh committed Mar 12, 2019
1 parent cf3e0c2 commit 5bf8a36
Showing 1 changed file with 10 additions and 5 deletions.
15 changes: 10 additions & 5 deletions app/scripts/lib/notification-manager.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
const extension = require('extensionizer')
const height = 620
const width = 360

const NOTIFICATION_HEIGHT = 620
const NOTIFICATION_WIDTH = 360

class NotificationManager {

Expand All @@ -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)
}
Expand Down

0 comments on commit 5bf8a36

Please sign in to comment.