luciano/reitit
1
0
Fork 0
mirror of https://github.com/metosin/reitit.git synced 2025-12-22 18:41:10 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Simplify contentEditable check for frontend routing

Browse source
This commit is contained in:
Juho Teperi 2019-02-08 13:57:03 +02:00
parent f5eb1d17f5
commit 6123230f98
2 changed files with 12 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
examples/frontend/src/frontend/core.cljs
View file

@ -27,7 +27,13 @@
[:ul
[:li [:a {:href "http://google.com"} "external link"]]
[:li [:a {:href (rfe/href ::foobar)} "Missing route"]]
[:li [:a {:href (rfe/href ::item)} "Missing route params"]]]])
[:li [:a {:href (rfe/href ::item)} "Missing route params"]]]
[:div
{:content-editable true
:suppressContentEditableWarning true}
[:p "Link inside contentEditable element is ignored."]
[:a {:href (rfe/href ::frontpage)} "Link"]]])
(defn item-page [match]
(let [{:keys [path query]} (:parameters match)

9
modules/reitit-frontend/src/reitit/frontend/history.cljs
View file

@ -4,8 +4,7 @@
(:require [reitit.core :as reitit]
[reitit.core :as r]
[reitit.frontend :as rf]
[goog.events :as gevents]
[goog.dom :as gdom])
[goog.events :as gevents])
(:import goog.Uri))
(defprotocol History
@ -78,8 +77,10 @@
(not (contains? #{"_blank" "self"} (.getAttribute el "target")))
;; Left button
(= 0 (.-button e))
(reitit/match-by-path router (.getPath uri))
(not (gdom/getAncestor el (fn [node] (.isContentEditable node)))))
;; isContentEditable property is inherited from parents,
;; so if the anchor is inside contenteditable div, the property will be true.
(not (.-isContentEditable (.-target e)))
(reitit/match-by-path router (.getPath uri)))
(.preventDefault e)
(let [path (str (.getPath uri)
(if (seq (.getQuery uri))