-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Performance improvements #20
Comments
This is a duplicate of issue #17. |
Regarding the separate class for typesetting, the generated CSS rule: |
OK, I've done more testing and I still can't find what the width problem was that the relative position was supposed to fix. It could be that a later change took care of it in another way (I'm afraid I can't keep track of all the IE bugs I've had to work around). I removed the processing class. Anything else need changing? |
It would be good to have a test page where |
OK, I think I have merged this in. (I'm still a little awkward with git, so I'm not sure I did it in the most efficient way, but the changes are now there.) So I'm closing the issue. |
I've made a performance branch of my fork that includes two updates: the reduced
processUpdateTime
for better responsiveness, and the separate class for during the typesetting that IE can use to set position:relative to get measurements right.See dpvc@f86e817
and dpvc@1e4efb3 for details.
Davide
The text was updated successfully, but these errors were encountered: