fix: add noopener to window.open fallback in saveImage
This commit is contained in:
@@ -57,7 +57,7 @@ describe('saveImage', () => {
|
|||||||
|
|
||||||
await saveImage('https://media.tenor.com/abc/tenor.gif');
|
await saveImage('https://media.tenor.com/abc/tenor.gif');
|
||||||
|
|
||||||
expect(openSpy).toHaveBeenCalledWith('https://media.tenor.com/abc/tenor.gif', '_blank');
|
expect(openSpy).toHaveBeenCalledWith('https://media.tenor.com/abc/tenor.gif', '_blank', 'noopener');
|
||||||
expect(mockAddToast).toHaveBeenCalledWith('Opened in new tab', 'info', 3000);
|
expect(mockAddToast).toHaveBeenCalledWith('Opened in new tab', 'info', 3000);
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -20,7 +20,7 @@ export async function saveImage(url: string, filename?: string): Promise<void> {
|
|||||||
a.remove();
|
a.remove();
|
||||||
URL.revokeObjectURL(blobUrl);
|
URL.revokeObjectURL(blobUrl);
|
||||||
} catch {
|
} catch {
|
||||||
window.open(url, '_blank');
|
window.open(url, '_blank', 'noopener');
|
||||||
useUIStore.getState().addToast('Opened in new tab', 'info', 3000);
|
useUIStore.getState().addToast('Opened in new tab', 'info', 3000);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user