• 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
.github Loading commit data...
bin Loading commit data...
dev-server Loading commit data...
docs Loading commit data...
embedding-examples Loading commit data...
images Loading commit data...
requirements Loading commit data...
scripts Loading commit data...
src Loading commit data...
.babelrc Loading commit data...
.dockerignore Loading commit data...
.eslintignore Loading commit data...
.eslintrc Loading commit data...
.gitignore Loading commit data...
.npmignore Loading commit data...
.npmrc Loading commit data...
.prettierignore Loading commit data...
.python-version Loading commit data...
CODE_OF_CONDUCT Loading commit data...
Dockerfile Loading commit data...
Jenkinsfile Loading commit data...
LICENSE Loading commit data...
Makefile Loading commit data...
README.md Loading commit data...
codecov.yml Loading commit data...
gulpfile.js Loading commit data...
package.json Loading commit data...
pyproject.toml Loading commit data...
requirements-dev.in Loading commit data...
tox.ini Loading commit data...
tsconfig.json Loading commit data...
yarn.lock Loading commit data...