[Webkit-unassigned] [Bug 175715] Speedometer: Update to modern React version
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Aug 22 01:01:48 PDT 2017
https://bugs.webkit.org/show_bug.cgi?id=175715
--- Comment #13 from Mathias Bynens <mathias at qiwi.be> ---
The issue is this: https://github.com/facebook/react/commit/045f1a791c6e17253e9d927ffca70ae5d00b4fe5 (which was later cherry-picked and backported to React v15) intentionally prevented React’s `onChange` from triggering after programmatic value changes.
As a result, our `handleChange` never gets called when using the runner, so the new TODO item’s internal value never gets updated through the `setState` call. It’s stuck at the default value, i.e. the empty string.
Therefore, when Enter is programmatically pressed and `handleNewTodoKeyDown` is called, the `if (val)` check fails, and the TODO item never gets added to the list.
I’ve asked if there’s a reasonable way to work around this behavior without patching our React copy: https://github.com/facebook/react/pull/5746#issuecomment-323947947
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170822/bad7797e/attachment.html>
More information about the webkit-unassigned
mailing list