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

Added setting to disable files in completion settings #1180

Closed
wants to merge 1 commit into from

Conversation

AlexXuChen
Copy link
Contributor

@AlexXuChen AlexXuChen commented Feb 28, 2022

Added setting to disable files in completion settings.

Part of redhat-developer/vscode-xml/issues/670

Signed-off-by: Alexander Chen [email protected]

@@ -26,13 +26,16 @@

private boolean autoCloseRemovesContent;

public XMLCompletionSettings(boolean autoCloseTags, boolean autoCloseRemovesContent) {
private boolean fileCompletion;
Copy link
Contributor

@angelozerr angelozerr Mar 1, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This settings belongs to the file path extension, so please don't add that in this class. I suggest that you see sample like https://github.com/eclipse/lemminx/blob/master/org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/extensions/contentmodel/settings/ContentModelSettings.java

In other words, you should create a settings/FilePathSettings.java in https://github.com/eclipse/lemminx/blob/master/org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/extensions/general

Please rename general to filepath to have org.eclipse.lemminx.extensions.filepath instead of org.eclipse.lemminx.extensions.general package and update too https://github.com/eclipse/lemminx/blob/1a70ccf0434354dee821e5e5f8111277378a57b4/org.eclipse.lemminx/src/main/resources/META-INF/services/org.eclipse.lemminx.services.extensions.IXMLExtension#L9

@angelozerr
Copy link
Contributor

Fixed with #1293

@angelozerr angelozerr closed this Oct 24, 2022
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

Successfully merging this pull request may close these issues.

2 participants