appsmith icon indicating copy to clipboard operation
appsmith copied to clipboard

[Task]: Remove usage of entity markers for peek overlay (perf improvement)

Open eco-monk opened this issue 2 years ago • 0 comments

Description

Reduce the amount of time taken when editing code in JsEditor. In some cases, we’ve seen the application slow down because of the way we handled text marking on hover. https://github.com/appsmithorg/appsmith/issues/21467

So we're moving away from text marking to AST based peek overlay.


  • [x] Handle AST parsing for JsObject and property pane separately.
  • [ ] Handle False hovers (hover highlight and ast parsing)
    • [ ] Handle JsKeywords (export default, return).
    • [ ] Filter object property definitions, local variables and function calls.
  • [ ] Handle cypress failures (if any)

eco-monk avatar May 08 '23 06:05 eco-monk