fix: sort LanguageServer tables in preferences

- Sort table in LanguageServerPreferencePage by content type
- Sort table in LoggingPreferencePage by language server name

Author: sebthom

org.eclipse.lsp4e/src/org/eclipse/lsp4e/ui/LanguageServerPreferencePage.java

@@ -28,6 +28,8 @@
 import org.eclipse.jface.viewers.IStructuredSelection;
 import org.eclipse.jface.viewers.TableViewer;
 import org.eclipse.jface.viewers.TableViewerColumn;
+import org.eclipse.jface.viewers.Viewer;
+import org.eclipse.jface.viewers.ViewerComparator;
 import org.eclipse.lsp4e.ContentTypeToLSPLaunchConfigEntry;
 import org.eclipse.lsp4e.ContentTypeToLanguageServerDefinition;
 import org.eclipse.lsp4e.LanguageServersRegistry;
@@ -252,6 +254,15 @@ public String getText(Object element) {
 			});
 		}
 
+		// Sort by content type label
+		checkboxViewer.setComparator(new ViewerComparator() {
+			@Override
+			public int compare(Viewer viewer, Object e1, Object e2) {
+				String s1 = ((ContentTypeToLanguageServerDefinition) e1).getKey().getName();
+				String s2 = ((ContentTypeToLanguageServerDefinition) e2).getKey().getName();
+				return getComparator().compare(s1, s2);
+			}
+		});
 		checkboxViewer.setInput(contentTypeToLanguageServerDefinitions);
 		checkboxViewer.getControl().setLayoutData(new GridData(SWT.FILL, SWT.FILL, true, true, 2, 1));
 		checkboxViewer.getTable().setHeaderVisible(true);

org.eclipse.lsp4e/src/org/eclipse/lsp4e/ui/LoggingPreferencePage.java

@@ -33,6 +33,8 @@
 import org.eclipse.jface.viewers.TableViewer;
 import org.eclipse.jface.viewers.TableViewerColumn;
 import org.eclipse.jface.wizard.WizardDialog;
+import org.eclipse.jface.viewers.Viewer;
+import org.eclipse.jface.viewers.ViewerComparator;
 import org.eclipse.lsp4e.ContentTypeToLSPLaunchConfigEntry;
 import org.eclipse.lsp4e.ContentTypeToLanguageServerDefinition;
 import org.eclipse.lsp4e.LanguageServerPlugin;
@@ -146,6 +148,17 @@ public String getText(Object element) {
 			}
 		});
 		addLoggingColumnsToViewer(languageServerViewer);
+
+		// Sort by language server label
+		languageServerViewer.setComparator(new ViewerComparator() {
+			@Override
+			public int compare(Viewer viewer, Object e1, Object e2) {
+				String s1 = ((ContentTypeToLanguageServerDefinition) e1).getValue().label;
+				String s2 = ((ContentTypeToLanguageServerDefinition) e2).getValue().label;
+				return getComparator().compare(s1, s2);
+			}
+		});
+
 		languageServerViewer.getControl().setLayoutData(new GridData(SWT.FILL, SWT.FILL, true, true));
 		languageServerViewer.getTable().setHeaderVisible(true);
 		languageServerViewer.getTable().setLinesVisible(true);