Merge pull request #306 from vector-im/bwindels/prevent-reload-on-notif-click

Fix reloading or opening wrong page when clicking notif
This commit is contained in:
Bruno Windels 2021-03-31 14:55:06 +00:00 committed by GitHub
commit 5a54eda512
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 2 deletions

View File

@ -76,6 +76,8 @@ export class ServiceWorkerHandler {
// this flag is read in fetch.js
this.haltRequests = true;
event.source.postMessage({replyTo: data.id});
} else if (data.type === "openRoom") {
this._navigation.push("room", data.payload.roomId);
}
}

View File

@ -196,13 +196,13 @@ async function openClientFromNotif(event) {
const {sessionId, roomId} = event.notification.data;
const sessionHash = `#/session/${sessionId}`;
const roomHash = `${sessionHash}/room/${roomId}`;
const roomURL = `/${roomHash}`;
const clientWithSession = await findClient(async client => {
return await sendAndWaitForReply(client, "hasSessionOpen", {sessionId});
});
if (clientWithSession) {
console.log("notificationclick: client has session open, showing room there");
clientWithSession.navigate(roomURL);
// use a message rather than clientWithSession.navigate here as this refreshes the page on chrome
clientWithSession.postMessage({type: "openRoom", payload: {roomId}});
if ('focus' in clientWithSession) {
try {
await clientWithSession.focus();
@ -210,6 +210,7 @@ async function openClientFromNotif(event) {
}
} else if (self.clients.openWindow) {
console.log("notificationclick: no client found with session open, opening new window");
const roomURL = new URL(`./${roomHash}`, baseURL).href;
await self.clients.openWindow(roomURL);
}
}