-
Notifications
You must be signed in to change notification settings - Fork 289
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
Fix Server Side rendering support using suggested PR from fixed-data-table #12
Fix Server Side rendering support using suggested PR from fixed-data-table #12
Conversation
* | ||
* This source code is licensed under the BSD-style license found in the | ||
* LICENSE file in the root directory of this source tree. An additional grant | ||
* of patent rights can be found in the PATENTS file in the same directory. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs to be updated to our new copyright
/**
* Copyright Schrodinger, LLC
*/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
LGTM after we verify this works with example provided here: #11 |
Verified it fixes the example. Also verified it still works as expected in LiveDesign. |
I spin up the dev server |
Done, everything looks good |
Description
have some state that tells us we are in the first pass and always render positions with left and top (since we know all browsers support this but not translate pos), then on any other render from there we can use the fancy translate3d stuff
Motivation and Context
Server side rendering currently produces different markup than client rendering
#11
How Has This Been Tested?
I haven't tried this out. I'll verify everything still works with client side rendering.
We should put together a server side example / sandbox for testing
Types of changes
Checklist: