New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Focusing; Timing #94

Merged
merged 1 commit into from Sep 8, 2017

Conversation

1 participant
@matatk
Owner

matatk commented Sep 8, 2017

  • Use the DOM to store elements' previous outline properties, so as to
    keep the state there and keep the code simple. It aims to be as clean
    to the DOM as can be. Fixes #54 (again, and properly :-)).
  • From studying how long pages take to get to the 'complete' readyState
    in practice, the previous number and interval of attempts was way too
    much: reduce the delay time to 500ms (still way above what seems to be
    necessary) and only make four attempts.
Focusing; Timing
* Use the DOM to store elements' previous outline properties, so as to
  keep the state there and keep the code simple. It aims to be as clean
  to the DOM as can be. Fixes #54 (again, and properly :-)).
* From studying how long pages take to get to the 'complete' readyState
  in practice, the previous number and interval of attempts was way too
  much: reduce the delay time to 500ms (still way above what seems to be
  necessary) and only make four attempts.

@matatk matatk merged commit ee710ea into master Sep 8, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@matatk matatk deleted the fix-style-restoration branch Sep 8, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment