Closed jamesdabbs closed 1 year ago
jamesdabbs
commented
3 years ago Nope, nothing specific.
the delay is about a second
I've noticed this too, and it's on my list. I'm not sure if the production build has the same issue, but I suspect (in either case) that it's something about katex rendering, and that we can defer that to happen in a tick after page load s.t. navigation stays responsive.
StevenClontz
commented
3 years ago It being a KaTeX thing sounds plausible.
Another noticeable performance issue is typing into a text box (which also happens in the current React site): there's often a delay between typing a character and it appearing in the box (seemingly tied to when the app filters through a large dataset). It does appear in the URL immediately though, so it's probably just another place where rendering needs to be delayed a moment.
cypress[bot]
commented
3 years ago
7 • 
0 • 
0 • 
0
| Project | viewer |
| Status | Passed |
| Commit | 90963e90b0 |
| Started | Nov 30, 2020 4:10 PM |
| Ended | Nov 30, 2020 4:10 PM |
| Duration | 00:18 💡 |
| OS | Linux Ubuntu - 18.04 |
| Browser | Electron 87  |
View run in Cypress Dashboard ➡️
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard
Currently with support for development / state management, and UI for properties only. This should be enough to be confident in the general approach. We'll iterate from here.
Obviously, this shouldn't be merged until it's at parity with the existing implementation, but we can use this as a long-running branch for that work.