-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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: Eliminate Uncaught TypeError in dev tools #4666
Conversation
✅ Deploy Preview for continuedev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
this looks great @lkk214 — can we also keep around the dev tools option by adding it to the apply?
@sestinj JS_QUERY_POOL_SIZE? It is just a configuration parameter in jbcef, and maybe it can be configured in the tools settings in JetBrains, but I don't think users will care about it. |
@@ -43,11 +43,6 @@ class ContinuePluginToolWindowFactory : ToolWindowFactory, DumbAware { | |||
class ContinuePluginWindow(project: Project) { | |||
private val defaultGUIUrl = "http://continue/index.html" | |||
|
|||
init { | |||
System.setProperty("ide.browser.jcef.jsQueryPoolSize", JS_QUERY_POOL_SIZE) | |||
System.setProperty("ide.browser.jcef.contextMenu.devTools.enabled", "true") |
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.
This is the line I'm referencing. It allows you to right click and "Open Dev Tools" which is very useful for debugging
@sestinj Oh, actually it has been an invalid configuration, it will only take effect when jcef is opened when starting the plugin, even if we don't configure it, we can still open it by right clicking and "Open Dev Tools". |
Description
Eliminate the
Uncaught TypeError: window.cefQuery_xxx is not a function
error that keeps appearing in the JS debug console.Screenshots