From dbb3d5c7c1336e8cff7fe571695853e8337ffb0e Mon Sep 17 00:00:00 2001 From: Dominik Picheta Date: Thu, 24 May 2018 14:15:53 +0100 Subject: [PATCH] Fixes Ctrl/Cmd not opening links in new tab. --- src/frontend/karaxutils.nim | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/src/frontend/karaxutils.nim b/src/frontend/karaxutils.nim index 9a0816e..462d000 100644 --- a/src/frontend/karaxutils.nim +++ b/src/frontend/karaxutils.nim @@ -77,13 +77,15 @@ when defined(js): # Fire the popState event. dom.dispatchEvent(dom.window, dom.newEvent("popstate")) - proc anchorCB*(e: kdom.Event, n: VNode) = # TODO: Why does this need disamb? - e.preventDefault() + proc anchorCB*(e: Event, n: VNode) = + let mE = e.MouseEvent + if not (mE.metaKey or mE.ctrlKey): + e.preventDefault() - # TODO: Why does Karax have it's own Node type? That's just silly. - let url = n.getAttr("href") + # TODO: Why does Karax have it's own Node type? That's just silly. + let url = n.getAttr("href") - navigateTo(url) + navigateTo(url) type FormData* = ref object