-
Notifications
You must be signed in to change notification settings - Fork 130
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
Custom commands #139
base: master
Are you sure you want to change the base?
Custom commands #139
Conversation
Hi. Thanks for the PR. There is a lot of changes to review, and I haven't looked at all of it. First thing I noticed is that you seem to have removed lazy loading of the popup. I think this goes against the first design principle. https://github.com/sorenkrabbe/Chrome-Salesforce-inspector#design-principles Because the popup runs a lot of JavaScript when it loads, which could slow other things down, and because it makes queries to the Salesforce API when it loads, which could then affect your Salesforce limits even when you are not using the inspector. |
Looks like you're right. Will fix it soon. |
@jesperkristensen returned back lazy loading. |
@jesperkristensen , @sorenkrabbe any way to merge it please? |
Is this functionality available yet? Is it possible to press a keyboard shortcut to open the Salesforce Inspector menu? |
Have you given up on this @jesperkristensen, @sorenkrabbe? Having the ability to open the SF Inspector panel using a keyboard shortcut is a simple but great improvement. |
@mhousaini-aware Have you tried those shortcut ? |
@tprouvot thanks for responding, that doesn't work. In your original comment, you suggested I've also noticed a few other issues in relation to the keyboard shortcuts, I'll list them here for your reference - from least to most important.
|
This allows user to set a hotkey to open/close salesforce inspector menu.
If merged, could be implemented to CtrlSpace in Data Export as well.