• Robert Knight's avatar
    Remove ThreadList debugging code · b4fe9177
    Robert Knight authored
    I believe we know what was causing the issue in ThreadList where an
    effect callback was sometimes called with closed-over variables which
    did not match the most recent render. See [1] for details. The issue
    really needs to be resolved in a more general way than trying to work
    around it in individual components.
    
    This commit removes code added for debugging this and also adds a check
    for a legitimate way (ie. without anything going wrong inside Preact)
    that the problem could occur, although it should never happen in the
    actual app as we never render `ThreadList` in a container that is
    disconnected from the document.
    
    [1] https://github.com/hypothesis/client/pull/3665#issuecomment-895857072
    b4fe9177
Name
Last commit
Last update
..
components Loading commit data...
config Loading commit data...
helpers Loading commit data...
services Loading commit data...
store Loading commit data...
test Loading commit data...
util Loading commit data...
cross-origin-rpc.js Loading commit data...
icons.js Loading commit data...
index.js Loading commit data...
markdown-commands.js Loading commit data...
media-embedder.js Loading commit data...
render-markdown.js Loading commit data...
search-client.js Loading commit data...
service-context.js Loading commit data...
websocket.js Loading commit data...