We're updating the issue view to help you get more done. 

Investigate handling notifications natively in the desktop app so we can enable context isolation.

Description

See discussion on this PR: https://github.com/mattermost/desktop/pull/900

Yuya on adding context isolation:

1 2 3 4 As far as I know, there is no easy workaround. webapp invokes notifications by itself though, to detect that event, I think there is no other way than Notification() constructor. The largest reason to extend Notification is that. If it's missing, the app can't activate the main window when a notification is clicked (by design of electron), also can't switch tabs to the server which invoked the notification. If there is a way to hook notification.onclick, it would be a workaround. On the other hand, by adding another websocket connection, the app can know that new messages have arrived. In that case, the app would be able to invoke notification by itself. However we would need to add a mechanism to suppress notifications in webapp.

QA Test Steps

None

Status

Assignee

Unassigned

Reporter

Christopher Speller

Epic Link

Fix versions

QA Testing Areas

Other (write in QA test steps)