-
Notifications
You must be signed in to change notification settings - Fork 19
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
Disable CSS Transitions #63
Comments
Currently a fixed timeout is set in the code: https://github.com/cburgmer/csscritic/blob/master/src/browser/browserRenderer.js#L36 Can you show an example for where the rendering is incorrect? So far my assumption was that the browsers all ignore transition and animations when rendering inside an SVG. If necessary, the HTML/CSS could be transformed, similarly to how :focus/:hover is supported: https://github.com/cburgmer/rasterizeHTML.js/blob/master/src/documentHelper.js#L10. |
I was planning to support a "snapshot" feature for transitions/animations at some point, so one can test different frames throughout an animation. In that notion you would require the 0% stage I believe. |
Testcase:
|
I can't reproduce with this example. Here's what I did: Registered your testcase via
Opened it in Chrome 47.0.2504.0 canary (64-bit). I also made the transition slower:
I would expect that I'd see the small boxes, but instead they are always at 100% width. Do I need to do something differently? |
Try a width of 200. I'm sure I saw it with that. Approve it and refresh.
|
I can't reproduce with a width of 200. I am testing on Mac OSX - in my experience the environment can make a big difference. |
I can finally reproduce on Chrome and Mac OSX. Did you happen to have a look at it in the meantime? |
I had a page that wouldn't render correctly on Chrome Stable, but would on Chrome Canary, until I realised that the css used transitons, and because I have less chrome extensions on canary that run JS on page load, it was able to complete the transition before it rasterized the page.
Would it be possible to have an option for disabling CSS transition. Perhaps something generic that could look for rules with certain props, values or selectors and ignoring them.
Alternatively, could there be a configurable delay before rasterization?
The text was updated successfully, but these errors were encountered: