Community
Participate
Working Groups
IMHO I think all extension points of generic editor (contentAssistProcessors, highlightReconcilers, etc) should support an <enabledWhen> in their extension point (like codemining does for instance). A usecase is with highlightReconcilers. The default word highlightReconcilers works well but if you install lsp4e, it doesn't work since lsp4e defines a highlightReconcilers for any content type. lsp4e should check if the content type has language server (or better if the language server supports highlight). In other words, lsp4e should do that: --------------------------------------------------------- <extension point="org.eclipse.ui.genericeditor.highlightReconcilers"> <highlightReconciler class="org.eclipse.lsp4e.operations.highlight.HighlightReconciler" contentType="org.eclipse.core.runtime.text"> <enabledWhen> <with variable="editorInput"> <test property="org.eclipse.lsp4e.hasLanguageServer"> </test> </with> </enabledWhen> </highlightReconciler> </extension> --------------------------------------------------------- After that, even after installing lsp4e, default word highlight will work with *.txt file for instance (today it doesn't work).
New Gerrit change created: https://git.eclipse.org/r/119994
*** Bug 514979 has been marked as a duplicate of this bug. ***
Keeping open as this deserve a note in the N&N
New Gerrit change created: https://git.eclipse.org/r/120252
> Keeping open as this deserve a note in the N&N Please review https://git.eclipse.org/r/120252
Gerrit change https://git.eclipse.org/r/119994 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=2c46f0a21f3a27b35898ca47d062ac7cf375c4b9
New Gerrit change created: https://git.eclipse.org/r/120262
New Gerrit change created: https://git.eclipse.org/r/120293
Gerrit change https://git.eclipse.org/r/120252 was merged to [master]. Commit: http://git.eclipse.org/c/www.eclipse.org/eclipse/news.git/commit/?id=2de7ca8dd2d88d9989f85b43e0d293334b761fe0
Gerrit change https://git.eclipse.org/r/120293 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=d6a4c8cfc2c8a4e690383ed530c5b706e3162441
New Gerrit change created: https://git.eclipse.org/r/120716
Gerrit change https://git.eclipse.org/r/120716 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.text.git/commit/?id=2137b10f5fa34ac875370405f798b8a8f3550f61