External URLs should open in system browser

[simonw]

For example the "powered by Datasette" link, or URLs in tables that get converted into links.

[simonw]

Relevant documentation: https://www.electronjs.org/docs/api/window-open

[simonw]

This recipe looks good: https://stackoverflow.com/a/67409223

mainWindow.webContents.setWindowOpenHandler(({ url }) => {
  shell.openExternal(url);
  return { action: 'deny' };
});

[simonw]

setWindowOpenHandler() only fires for target="_blank" or window.open() - I need to fire for all navigated links. This recipe seems to work:

function postConfigure(mainWindow) {
  mainWindow.webContents.on("will-navigate", function (event, reqUrl) {
    let requestedHost = new url.URL(reqUrl).host;
    let currentHost = new url.URL(mainWindow.webContents.getURL()).host;
    if (requestedHost && requestedHost != currentHost) {
      event.preventDefault();
      shell.openExternal(reqUrl);
    }
  });
}