status-im/reagent
2
0
Fork 0
mirror of https://github.com/status-im/reagent.git synced 2025-02-05 16:44:25 +00:00
Code Issues Projects Releases Wiki Activity

Remove "number" and "email" from the list of input types supporting selection API

Browse Source
This commit is contained in:
mike-thompson-day8 2015-05-12 09:04:22 +10:00
parent 985641d847
commit 6319a968ac
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/reagent/impl/template.cljs
View File

@ -91,11 +91,13 @@
(.! this :cljsInputValue nil))
;; <input type="??" >
(def these-inputs-have-a-cursor #{"text" "textarea" "password" "email" "number" "search" "tel" "url"})
;; The properites 'selectionStart' and 'selectionEnd' only exist on some inputs
;; See: https://html.spec.whatwg.org/multipage/forms.html#do-not-apply
(def these-inputs-have-selection-api #{"text" "textarea" "password" "search" "tel" "url"})
(defn has-cursor?
(defn has-selection-api?
[input-type]
(contains? these-inputs-have-a-cursor input-type))
(contains? these-inputs-have-selection-api input-type))
(defn input-set-value [this]
(when-some [value (.' this :cljsInputValue)]
@ -103,7 +105,7 @@
(let [node (.' this getDOMNode)
node-value (.' node :value)]
(when (not= value node-value)
(if-not (has-cursor? (.' node :type))
(if-not (has-selection-api? (.' node :type))
; just set the value, no need to worry about a cursor
(.! node :value value)