Rom Grk

Results 558 comments of Rom Grk

I have been spending way too much time researching hashing functions so here is some data. ### Performance First, here are the performance results across engines. The `_unmodified` variants are...

Funny thought I had but I took the [fast hex formatting](https://github.com/romgrk/js-benchmark/blob/504d5ef61065cb40a80a8b2e56cedbf36a425ac2/benchmarks/hashing.js#L210-L221) function from https://github.com/mui/material-ui/pull/43289 and replaced the final `n.toString(36)` by `formatHex(n)`, substantial performance improvement as well. So @Andarist @emmatown if...

Does this really need to be added? The `updateColumns` method is the recommended way to update large number of columns, React props don't let us make the same optimizations.

The environment info isn't filled. Please refer to the new issue instructions to include env info. Please also include the hardware you're using so we have an idea what kind...

@ceisele-r By any chance would you be able to build an unminified production version of your app, and record a stack trace with that? I've looked at the trace you...

The problem is the demo code, not the grid. MUI Material styling code is fairly slow and we've been working on mitigating that issue with the Emotion renderer, but indeed...

We should probably re-write those renderers, the demo datasets are used at many places in the docs and give a bad impression of the grid's performance.

If you have a problem with your own code, please post a reproducible example that showcases the performance issue. Your initial report only mentions the docs demo code which is...

Missed that. @Vagizova Please open a separate issue unless your issue is strictly the same. @ceisele-r As I mentioned above, the root cause of the docs demo issue is not...

This is an area where you would benefit from doing things outside of react. The tooltip position update could easily be a vanilla JS event handler that updates its position...