Skip to content
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

Is ProvideAutoLoad needed? #16

Open
madskristensen opened this issue Jul 28, 2021 · 3 comments
Open

Is ProvideAutoLoad needed? #16

madskristensen opened this issue Jul 28, 2021 · 3 comments

Comments

@madskristensen
Copy link
Contributor

When the commands are being invoked, the package is being loaded and the handlers initialized. Does this package need autoloading at all?

@justcla
Copy link
Owner

justcla commented Jul 28, 2021

I'm not sure.
There were several UI components: including a right click menu that is added to the editor margin. I think I just need to register the TextViewCreationListener. I also added a button to the Solution Explorer, but that might just be loaded by the VSCT.
I'll take a look.

@justcla
Copy link
Owner

justcla commented Jul 28, 2021

I looked into it. You're right; I don't need to autoload for the context menu items to show and work.
But the TrackActiveItems button that I added to the Soln Expl will show as OFF until it is pressed, regardless of the true state.
So if I don't auto-load, there will be a display bug in the TrackActiveItems button.

Got any suggestions?

@madskristensen
Copy link
Contributor Author

you probably need autoload then

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants