-
Notifications
You must be signed in to change notification settings - Fork 278
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
Matrix3d #136
base: master
Are you sure you want to change the base?
Matrix3d #136
Conversation
Syncing Lime Master to my repo
…rix3d Failed Commit message modification
This will take me some time to dig through. Can you clarify what problems this solves for you? I'm ok with the removal of the css transitions as long as there isn't a performance regression(iOS needs to be tested). That implementation is a mess anyway. Same goes for the setting transforms with matrix function. It makes code cleaner, so if its not slower I'm ok with it. Theres something similar in the webgl branch. I think then it was a bit slower but that was many years ago. Let me know when you think this is ready for merging. If you can, then also rebase it and squash the commits with same message. Or I can do it for you before the merge. |
Can you explain why the renderer does 2 passes? |
The second pass if for the renderer itself to setup post-processing that needs to happen during the event loop. For example on canvas render first pass sets up the layout and figures out where the Canvas elements need to go. Then it also registers a second pass for the element that has the canvas element and the second pass does the actual drawing. The drawing could not start before update has run for the child nodes, for example we could not even determine the size of the canvas without it. Something similar also happens for the masks and maybe for the transitions(but they are gone anyway in this PR). Basically it's just the order for the rendering tasks. Every time you change a node that starts a rendering change lime will call |
I've updated this code for more optimizations. I would like to add a table for arctan, but I am not sure how to do that yet. |
…ave so much precision that further operations cause them to goto Scientific Notation and that's can't be used in the matrix3d style element.
I added the remove-container-node fix here which fixes borders. I also embedded the trig-table values instead of calculating them. I still want to create a table for atan. |
Beginning work on the renderer. I am wondering how you feel about these changes. This is based on the work from Famo.US. I will be optimizing this more.