improve completion right alignment

ajaxorg · May 11, 2023 · b55a42b · b55a42b
1 parent 0472d54
commit b55a42b
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/autocomplete/popup.js b/src/autocomplete/popup.js
@@ -201,6 +201,7 @@ class AcePopup {
             }
             addToken(caption.slice(lastIndex, caption.length), "");
 
+            tokens.push({type: "completion-spacer", value: " "});
             if (data.meta)
                 tokens.push({type: "completion-meta", value: data.meta});
             if (data.message)
@@ -236,7 +237,7 @@ class AcePopup {
             return selectionMarker.start.row;
         };
         popup.setRow = function(line) {
-            line = Math.max(this.autoSelect ? 0 : -1, Math.min(this.data.length, line));
+            line = Math.max(this.autoSelect ? 0 : -1, Math.min(this.data.length - 1, line));
             if (selectionMarker.start.row != line) {
                 popup.selection.clearSelection();
                 selectionMarker.start.row = selectionMarker.end.row = line || 0;
@@ -441,9 +442,8 @@ dom.importCssString(`
 .ace_autocomplete_right .ace_line {
     display: flex;
 }
-.ace_autocomplete_right .ace_completion-meta {
+.ace_autocomplete_right .ace_completion-spacer {
     flex: 1;
-    text-align: right;
 }
 `, "autocompletion.css", false);