WIP fix partial auto-complete behavior and sort

Sort autocompletions, since right now we don't have any heuristic ranking for
them.

Fix autocompletion insertion behavior after partial auto-insertion.

Author: garfieldnate

src/main/java/edu/umich/soar/visualsoar/ruleeditor/RuleEditor.java

@@ -1777,6 +1777,8 @@ private void valueComplete(int pos, String userType, String prodSoFar) {
           }
         }
       }
+
+      Collections.sort(completeMatches);
       complete(pos, userType, completeMatches);
     } catch (ParseException pe) {
       getToolkit().beep();
@@ -1816,11 +1818,12 @@ private void complete(int pos, String userType, List<String> completeMatches) {
         ++curPos;
       }
     }
-    // TODO: hmmm, no I don't think so, let the user choose what to insert
+    // TODO: hmmm, if the user decides they don't want the completion after all, then they have to undo. Not super smooth.
     EditingUtils.insert(editorPane.getDocument(), addedCharacters, pos);
+    editorPane.colorSyntax();
 
     //report all matches to the user
-    RuleEditor.this.showAutocompletePopup(pos, userType, completeMatches);
+    RuleEditor.this.showAutocompletePopup(pos + addedCharacters.length(), userType + addedCharacters, completeMatches);
     if(completeMatches.isEmpty()) {
       MainFrame.getMainFrame().getFeedbackManager().setStatusBarMsg("No auto-complete matches found.");
     }
@@ -1830,12 +1833,8 @@ private void complete(int pos, String userType, List<String> completeMatches) {
 
   private void attributeComplete(int pos, String userType, String prodSoFar) {
     List<String> completeMatches = getMatchingStrings(userType, prodSoFar);
-    if (completeMatches.isEmpty()) {
-      return;
-    }
     complete(pos, userType, completeMatches);
-
-  }//attributeComplete
+  }
 
   private AutocompletePopup autocompletePopup = null;
 
@@ -1927,6 +1926,7 @@ private List<String> getMatchingStrings(String userType, String prodSoFar) {
           completeMatches.add(matched);
         }
       }
+      Collections.sort(completeMatches);
         return completeMatches;
     }