vndb/mithril-vndb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix #1252, re-fix #348

Browse source
This commit is contained in:
Pierre-Yves Gerardy 2016-08-18 01:05:16 +02:00
parent 0039a0b4a5
commit 51bfd511d2
1 changed files with 12 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

15
mithril.js
View file

@ -1081,8 +1081,14 @@
// //
// #348 don't set the value if not needed - otherwise, cursor // #348 don't set the value if not needed - otherwise, cursor
// placement breaks in Chrome // placement breaks in Chrome
// #1252 likewise when `contenteditable` is set on an element.
try { try {
if (tag !== "input" || node[attrName] !== dataAttr) { if (
tag !== "input" && !node.isContentEditable
/* eslint-disable eqeqeq */
|| node[attrName] != dataAttr
/* eslint-enable eqeqeq */
) {
node[attrName] = dataAttr node[attrName] = dataAttr
} }
} catch (e) { } catch (e) {
@ -1119,8 +1125,11 @@
if (e.message.indexOf("Invalid argument") < 0) throw e if (e.message.indexOf("Invalid argument") < 0) throw e
} }
} else if (attrName === "value" && tag === "input" && } else if (attrName === "value" && tag === "input" &&
node.value !== dataAttr) { /* eslint-disable eqeqeq */
// #348 dataAttr may not be a string, so use loose comparison node.value != dataAttr) {
// #348 dataAttr may not be a string,
// so use loose comparison
/* eslint-enable eqeqeq */
node.value = dataAttr node.value = dataAttr
} }
} }