Merge pull request #75 from LeanBitLab/rename-popup-specifiers-5914839074087632464

LeanBitLab · web-flow · commit aa788aa51963 · 2026-04-28T01:22:31.000+05:30
Refactor: Rename popup keys auto and fixed column specifiers
diff --git a/app/src/main/assets/layouts/main/bengali_baishakhi.json b/app/src/main/assets/layouts/main/bengali_baishakhi.json
@@ -94,7 +94,7 @@
       },
       { "$": "shift_state_selector",
         "manualOrLocked": { "label": "ঁ", "labelFlags": 1073741824, "popup": { "relevant": [
-            { "label": "!autoColumnOrder!6" },
+            { "label": "!autoOrder!6" },
             { "label": "়" },
             { "label": "ৄ" },
             { "label": "ঽ" },
diff --git a/app/src/main/assets/layouts/main/bengali_inscript.json b/app/src/main/assets/layouts/main/bengali_inscript.json
@@ -98,7 +98,7 @@
     },
     { "$": "shift_state_selector",
       "manualOrLocked": { "label": "ঁ", "labelFlags": 1073741824, "popup": { "relevant": [
-            {"label": "!autoColumnOrder!6" },
+            {"label": "!autoOrder!6" },
             { "label": "়" },
             { "label": "ৄ" },
             { "label": "ঽ" },
diff --git a/app/src/main/assets/layouts/main/bengali_probhat.json b/app/src/main/assets/layouts/main/bengali_probhat.json
@@ -122,7 +122,7 @@
     },
     { "$": "shift_state_selector",
       "manualOrLocked": { "label": "ঁ", "popup": { "relevant": [
-            { "label": "!autoColumnOrder!6"},
+            { "label": "!autoOrder!6"},
             { "label": "়" },
             { "label": "ৄ"},
             { "label": "ঽ"},
diff --git a/app/src/main/assets/layouts/main/bengali_unijoy.json b/app/src/main/assets/layouts/main/bengali_unijoy.json
@@ -60,7 +60,7 @@
     },
     { "$": "shift_state_selector",
       "manualOrLocked": { "label": "ঁ", "labelFlags": 1073741824, "popup": { "relevant": [
-            { "label": "!autoColumnOrder!6" },
+            { "label": "!autoOrder!6" },
             { "label": "়" },
             { "label": "ৄ" },
             { "label": "ঽ" },
diff --git a/app/src/main/assets/layouts/more_symbols/symbols_shifted.txt b/app/src/main/assets/layouts/more_symbols/symbols_shifted.txt
@@ -27,5 +27,5 @@ $$$4
 [
 ]
 
-< !fixedColumnOrder!3 ‹ ≤ «
-> !fixedColumnOrder!3 › ≥ »
+< !fixedOrder!3 ‹ ≤ «
+> !fixedOrder!3 › ≥ »
diff --git a/app/src/main/assets/layouts/number/number.json b/app/src/main/assets/layouts/number/number.json
@@ -93,7 +93,7 @@
         "popup": {
           "relevant": [
             {
-              "label": "!fixedColumnOrder!2"
+              "label": "!fixedOrder!2"
             },
             {
               "label": "!hasLabels!"
@@ -115,7 +115,7 @@
         "popup": {
           "relevant": [
             {
-              "label": "!fixedColumnOrder!2"
+              "label": "!fixedOrder!2"
             },
             {
               "label": "!hasLabels!"
diff --git a/app/src/main/assets/locale_key_texts/ar.txt b/app/src/main/assets/locale_key_texts/ar.txt
@@ -7,12 +7,12 @@
 ي ئ ى
 ب پ
 ل ﻻ|لا ﻷ|لأ ﻹ|لإ ﻵ|لآ
-ا !fixedColumnOrder!5 آ ء أ إ ٱ
+ا !fixedOrder!5 آ ء أ إ ٱ
 ك گ ک
 ى ئ
 ز ژ
 و ؤ
-punctuation !fixedColumnOrder!7  ٕ|ٕ  ٔ|ٔ  ْ|ْ  ٍ|ٍ  ٌ|ٌ  ً|ً  ّ|ّ  ٖ|ٖ  ٰ|ٰ  ٓ|ٓ  ِ|ِ  ُ|ُ  َ|َ ـــ|ـ
+punctuation !fixedOrder!7  ٕ|ٕ  ٔ|ٔ  ْ|ْ  ٍ|ٍ  ٌ|ٌ  ً|ً  ّ|ّ  ٖ|ٖ  ٰ|ٰ  ٓ|ٓ  ِ|ِ  ُ|ُ  َ|َ ـــ|ـ
 « „ “ ”
 » ‚ ‘ ’ ‹ ›
 
diff --git a/app/src/main/assets/locale_key_texts/bn-BD.txt b/app/src/main/assets/locale_key_texts/bn-BD.txt
@@ -16,7 +16,7 @@
 ী ঈ
 া আ অ
 ্ ঁ
-ঁ !autoColumnOrder!6 ় ৄ ঽ ৢ ৱ ৣ ৗ ৠ ৺ ঌ ৰ ৡ
+ঁ !autoOrder!6 ় ৄ ঽ ৢ ৱ ৣ ৗ ৠ ৺ ঌ ৰ ৡ
 ব ভ
 ক খ
 ত থ ৎ
@@ -30,7 +30,7 @@
 ন ণ
 স ষ
 ম শ
-punctuation !autoColumnOrder!8 \, ॥ ? ! !icon/zwnj_key|‌ !icon/zwj_key|‍ # @ ( ) / ; : - + \%
+punctuation !autoOrder!8 \, ॥ ? ! !icon/zwnj_key|‌ !icon/zwj_key|‍ # @ ( ) / ; : - + \%
 
 [labels]
 alphabet: কখগ
diff --git a/app/src/main/assets/locale_key_texts/bn-IN.txt b/app/src/main/assets/locale_key_texts/bn-IN.txt
@@ -23,14 +23,14 @@
 ট ঠ
 ৃ ঋ
 ং ঁ ঃ
-ঁ !autoColumnOrder!6 ় ৄ ঽ ৢ ৱ ৣ ৗ ৠ ৺ ঌ ৰ ৡ
+ঁ !autoOrder!6 ় ৄ ঽ ৢ ৱ ৣ ৗ ৠ ৺ ঌ ৰ ৡ
 ম ণ
 ন ঞ
 ব ঢ়
 ল ষ
 স শ
 য় য
-punctuation !autoColumnOrder!8 \, ॥ ? ! !icon/zwnj_key|‌ !icon/zwj_key|‍ # @ ( ) / ; : - + \%
+punctuation !autoOrder!8 \, ॥ ? ! !icon/zwnj_key|‌ !icon/zwj_key|‍ # @ ( ) / ; : - + \%
 
 [labels]
 alphabet: কখগ
diff --git a/app/src/main/assets/locale_key_texts/ca.txt b/app/src/main/assets/locale_key_texts/ca.txt
@@ -6,7 +6,7 @@ o ò ó
 u ú ü
 c ç
 l l·l
-punctuation !autoColumnOrder!9 \, ? ! · # ) ( / ; ' @ : - " + \% &
+punctuation !autoOrder!9 \, ? ! · # ) ( / ; ' @ : - " + \% &
 
 [extra_keys]
 2: ç
diff --git a/app/src/main/assets/locale_key_texts/ckb.txt b/app/src/main/assets/locale_key_texts/ckb.txt
@@ -22,7 +22,7 @@
 ز ظ
 ع ؏
 ب ى 
-punctuation !autoColumnOrder!8 \؟ ! ، ٫ ؍ : ؛ ; : | - @ _ # * ٪ & ^
+punctuation !autoOrder!8 \؟ ! ، ٫ ؍ : ؛ ; : | - @ _ # * ٪ & ^
 « „ “ ”
 » ‚ ‘ ’ ‹ ›
 
diff --git a/app/src/main/assets/locale_key_texts/es.txt b/app/src/main/assets/locale_key_texts/es.txt
@@ -6,7 +6,7 @@ o ó º
 u ú ü
 n ñ
 y ý
-punctuation !autoColumnOrder!9 \, ? ! # ) ( / ; ¡ ' @ : - " + \% & ¿
+punctuation !autoOrder!9 \, ? ! # ) ( / ; ¡ ' @ : - " + \% & ¿
 
 [extra_keys]
 2: ñ
diff --git a/app/src/main/assets/locale_key_texts/fa.txt b/app/src/main/assets/locale_key_texts/fa.txt
@@ -1,11 +1,11 @@
 [popup_keys]
 ه ﻫ|ه‍ هٔ ة
 ی ئ ي ﯨ|ى
-ا !fixedColumnOrder!5 ٱ ء آ أ إ
+ا !fixedOrder!5 ٱ ء آ أ إ
 ت ة
 ک ك
 و ؤ
-punctuation !fixedColumnOrder!7  ٕ|ٕ  ْ|ْ  ّ|ّ  ٌ|ٌ  ٍ|ٍ  ً|ً  ٔ|ٔ  ٖ|ٖ  ٰ|ٰ  ٓ|ٓ  ُ|ُ  ِ|ِ  َ|َ ـــ|ـ
+punctuation !fixedOrder!7  ٕ|ٕ  ْ|ْ  ّ|ّ  ٌ|ٌ  ٍ|ٍ  ً|ً  ٔ|ٔ  ٖ|ٖ  ٰ|ٰ  ٓ|ٓ  ُ|ُ  ِ|ِ  َ|َ ـــ|ـ
 « „ “ ”
 » ‚ ‘ ’ ‹ ›
 
diff --git a/app/src/main/assets/locale_key_texts/hi.txt b/app/src/main/assets/locale_key_texts/hi.txt
@@ -37,7 +37,7 @@
 ल ऌ ॡ
 य य़
 ़ ॽ ॰ ऽ
-punctuation !autoColumnOrder!9 \, . ॥ ? ! !icon/zwnj_key|‌ !icon/zwj_key|‍ # @ ( ) / ; : - + \%
+punctuation !autoOrder!9 \, . ॥ ? ! !icon/zwnj_key|‌ !icon/zwj_key|‍ # @ ( ) / ; : - + \%
 औ ौ
 ऐ ै
 आ ा
diff --git a/app/src/main/assets/locale_key_texts/hy.txt b/app/src/main/assets/locale_key_texts/hy.txt
@@ -1,5 +1,5 @@
 [popup_keys]
-punctuation !autoColumnOrder!8 \, ՞ ՜ ․ … ' = / ՝ ՛ ֊ » « ― ) (
+punctuation !autoOrder!8 \, ՞ ՜ ․ … ' = / ՝ ՛ ֊ » « ― ) (
 ? ՞ ¿
 ! ՜ ¡
 
diff --git a/app/src/main/assets/locale_key_texts/more_popups_all.txt b/app/src/main/assets/locale_key_texts/more_popups_all.txt
@@ -11,7 +11,7 @@ g ğ ģ ġ g\' ĝ
 c ç ć č ċ ĉ
 z ž ź ż
 l ł ĺ ļ ľ ŀ l·l
-punctuation !autoColumnOrder!10 \, ? ! # ) ( / ; ' @ : - " + \% & · ¡ ¿
+punctuation !autoOrder!10 \, ? ! # ) ( / ; ' @ : - " + \% & · ¡ ¿
 d ď đ ð
 r ř ŕ ŗ
 t ť ţ þ ț ŧ
diff --git a/app/src/main/assets/locale_key_texts/more_popups_more.txt b/app/src/main/assets/locale_key_texts/more_popups_more.txt
@@ -11,7 +11,7 @@ g ğ ģ ġ
 c ç ć č ċ
 z ž ź ż
 l ł ĺ ļ ľ ŀ
-punctuation !autoColumnOrder!9 \, ? ! # ) ( / ; ' @ : - " + \% & ¡ ¿
+punctuation !autoOrder!9 \, ? ! # ) ( / ; ' @ : - " + \% & ¡ ¿
 d ď đ ð
 r ř ŕ ŗ
 t ť ţ þ ț ŧ
diff --git a/app/src/main/assets/locale_key_texts/my.txt b/app/src/main/assets/locale_key_texts/my.txt
@@ -1,5 +1,5 @@
 [popup_keys]
-punctuation !autoColumnOrder!9 ၊ . ? ! # ) ( / ; ... ' @ : - " + \% &
+punctuation !autoOrder!9 ၊ . ? ! # ) ( / ; ... ' @ : - " + \% &
 
 [labels]
 alphabet: ကခဂ
diff --git a/app/src/main/assets/locale_key_texts/ne.txt b/app/src/main/assets/locale_key_texts/ne.txt
@@ -1,7 +1,7 @@
 [popup_keys]
 ट ़
 ् ऽ
-punctuation !autoColumnOrder!9 . \, ? ! # ) ( / ; ' @ : - " + \% &
+punctuation !autoOrder!9 . \, ? ! # ) ( / ; ' @ : - " + \% &
 त्त ञ ज्ञ ॥
 ड्ढ ई
 ऐ घ
diff --git a/app/src/main/java/helium314/keyboard/keyboard/Key.java b/app/src/main/java/helium314/keyboard/keyboard/Key.java
@@ -145,9 +145,8 @@ public class Key implements Comparable<Key> {
     private static final int POPUP_KEYS_FLAGS_HAS_LABELS = 0x40000000;
     private static final int POPUP_KEYS_FLAGS_NEEDS_DIVIDERS = 0x20000000;
     private static final int POPUP_KEYS_FLAGS_NO_PANEL_AUTO_POPUP_KEY = 0x10000000;
-    // TODO: Rename these specifiers to !autoOrder! and !fixedOrder! respectively.
-    public static final String POPUP_KEYS_AUTO_COLUMN_ORDER = "!autoColumnOrder!";
-    public static final String POPUP_KEYS_FIXED_COLUMN_ORDER = "!fixedColumnOrder!";
+    public static final String POPUP_KEYS_AUTO_ORDER = "!autoOrder!";
+    public static final String POPUP_KEYS_FIXED_ORDER = "!fixedOrder!";
     public static final String POPUP_KEYS_HAS_LABELS = "!hasLabels!";
     private static final String POPUP_KEYS_NEEDS_DIVIDERS = "!needsDividers!";
     private static final String POPUP_KEYS_NO_PANEL_AUTO_POPUP_KEY = "!noPanelAutoPopupKey!";
@@ -1111,12 +1110,12 @@ private static int getPopupKeysColumnAndFlagsAndSetNullInArray(final KeyboardPar
             int popupKeysColumnAndFlags = POPUP_KEYS_MODE_MAX_COLUMN_WITH_AUTO_ORDER
                     | params.mMaxPopupKeysKeyboardColumn;
             int value;
-            if ((value = PopupKeySpec.getIntValue(popupKeys, POPUP_KEYS_AUTO_COLUMN_ORDER, -1)) > 0) {
+            if ((value = PopupKeySpec.getIntValue(popupKeys, POPUP_KEYS_AUTO_ORDER, -1)) > 0) {
                 // Override with fixed column order number and set a relevant mode value.
                 popupKeysColumnAndFlags = POPUP_KEYS_MODE_FIXED_COLUMN_WITH_AUTO_ORDER
                         | (value & POPUP_KEYS_COLUMN_NUMBER_MASK);
             }
-            if ((value = PopupKeySpec.getIntValue(popupKeys, POPUP_KEYS_FIXED_COLUMN_ORDER, -1)) > 0) {
+            if ((value = PopupKeySpec.getIntValue(popupKeys, POPUP_KEYS_FIXED_ORDER, -1)) > 0) {
                 // Override with fixed column order number and set a relevant mode value.
                 popupKeysColumnAndFlags = POPUP_KEYS_MODE_FIXED_COLUMN_WITH_FIXED_ORDER
                         | (value & POPUP_KEYS_COLUMN_NUMBER_MASK);
diff --git a/app/src/main/java/helium314/keyboard/keyboard/internal/keyboard_parser/LocaleKeyboardInfos.kt b/app/src/main/java/helium314/keyboard/keyboard/internal/keyboard_parser/LocaleKeyboardInfos.kt
@@ -53,15 +53,15 @@ class LocaleKeyboardInfos(dataStream: InputStream?, locale: Locale) {
         // should this also be done with punctuation popupKeys?
         // todo: those defaults should not be in here
         if ("\'" !in popupKeys)
-            popupKeys["\'"] = mutableListOf("!fixedColumnOrder!5", "‚", "‘", "’", "‹", "›")
+            popupKeys["\'"] = mutableListOf("!fixedOrder!5", "‚", "‘", "’", "‹", "›")
         if ("\"" !in popupKeys)
-            popupKeys["\""] = mutableListOf("!fixedColumnOrder!5", "„", "“", "”", "«", "»")
+            popupKeys["\""] = mutableListOf("!fixedOrder!5", "„", "“", "”", "«", "»")
         if ("!" !in popupKeys)
             popupKeys["!"] = mutableListOf("¡")
         if (labelQuestion !in popupKeys)
             popupKeys[labelQuestion] = if (labelQuestion == "?") mutableListOf("¿") else mutableListOf("?", "¿")
         if ("punctuation" !in popupKeys)
-            popupKeys["punctuation"] = mutableListOf("${Key.POPUP_KEYS_AUTO_COLUMN_ORDER}8", "\\,", "?", "!", "#", ")", "(", "/", ";", "'", "@", ":", "-", "\"", "+", "\\%", "&")
+            popupKeys["punctuation"] = mutableListOf("${Key.POPUP_KEYS_AUTO_ORDER}8", "\\,", "?", "!", "#", ")", "(", "/", ";", "'", "@", ":", "-", "\"", "+", "\\%", "&")
     }
 
     private fun readStream(stream: InputStream?, onlyPopupKeys: Boolean, priority: Boolean) {
@@ -170,21 +170,21 @@ class LocaleKeyboardInfos(dataStream: InputStream?, locale: Locale) {
 
 private fun addFixedColumnOrder(popupKeys: MutableCollection<String>) {
     // use intermediate list, because we can't add first in a LinkedHashSet (i.e. MutableSet)
-    popupKeys.removeAll { it.startsWith(Key.POPUP_KEYS_FIXED_COLUMN_ORDER) }
+    popupKeys.removeAll { it.startsWith(Key.POPUP_KEYS_FIXED_ORDER) }
     val temp = popupKeys.toList()
     popupKeys.clear()
-    popupKeys.add("${Key.POPUP_KEYS_FIXED_COLUMN_ORDER}${temp.size}")
+    popupKeys.add("${Key.POPUP_KEYS_FIXED_ORDER}${temp.size}")
     popupKeys.addAll(temp)
 }
 
 private fun adjustAutoColumnOrder(popupKeys: MutableCollection<String>) {
     // same style as above
-    // currently, POPUP_KEYS_AUTO_COLUMN_ORDER is only used for 2 lines of punctuation popups, so assume 2 lines
-    if (!popupKeys.removeAll { it.startsWith(Key.POPUP_KEYS_AUTO_COLUMN_ORDER) })
+    // currently, POPUP_KEYS_AUTO_ORDER is only used for 2 lines of punctuation popups, so assume 2 lines
+    if (!popupKeys.removeAll { it.startsWith(Key.POPUP_KEYS_AUTO_ORDER) })
         return
     val temp = popupKeys.toList()
     popupKeys.clear()
-    popupKeys.add("${Key.POPUP_KEYS_AUTO_COLUMN_ORDER}${((temp.size + 1) / 2).coerceAtMost(9)}")
+    popupKeys.add("${Key.POPUP_KEYS_AUTO_ORDER}${((temp.size + 1) / 2).coerceAtMost(9)}")
     popupKeys.addAll(temp)
 }
 
diff --git a/app/src/main/java/helium314/keyboard/keyboard/internal/keyboard_parser/floris/TextKeyData.kt b/app/src/main/java/helium314/keyboard/keyboard/internal/keyboard_parser/floris/TextKeyData.kt
@@ -126,9 +126,9 @@ sealed interface KeyData : AbstractKeyData {
                 // this makes use of removal of empty popupKeys in PopupKeySpec.insertAdditionalPopupKeys
                 popupKeys[popupKeys.indexOf("!")] = ""
                 popupKeys[popupKeys.indexOf("?")] = ""
-                val columns = popupKeys[0].substringAfter(Key.POPUP_KEYS_AUTO_COLUMN_ORDER).toIntOrNull()
+                val columns = popupKeys[0].substringAfter(Key.POPUP_KEYS_AUTO_ORDER).toIntOrNull()
                 if (columns != null)
-                    popupKeys[0] = "${Key.POPUP_KEYS_AUTO_COLUMN_ORDER}${columns - 1}"
+                    popupKeys[0] = "${Key.POPUP_KEYS_AUTO_ORDER}${columns - 1}"
             }
             return popupKeys
         }
@@ -208,9 +208,9 @@ sealed interface KeyData : AbstractKeyData {
             // (probably not necessary, but whatever) and in emoji mode
             if ((Settings.getInstance().isTablet || params.mId.mElementId == KeyboardId.ELEMENT_EMOJI_BOTTOM_ROW)
                 && popupKeys.remove("!icon/emoji_action_key|!code/key_emoji")) {
-                val i = popupKeys.indexOfFirst { it.startsWith(Key.POPUP_KEYS_FIXED_COLUMN_ORDER) }
+                val i = popupKeys.indexOfFirst { it.startsWith(Key.POPUP_KEYS_FIXED_ORDER) }
                 if (i > -1) {
-                    val n = popupKeys[i].substringAfter(Key.POPUP_KEYS_FIXED_COLUMN_ORDER).toIntOrNull()
+                    val n = popupKeys[i].substringAfter(Key.POPUP_KEYS_FIXED_ORDER).toIntOrNull()
                     if (n != null)
                         popupKeys[i] = popupKeys[i].replace(n.toString(), (n - 1).toString())
                 }
@@ -245,11 +245,11 @@ sealed interface KeyData : AbstractKeyData {
         // could make arrays right away, but they need to be copied anyway as popupKeys arrays are modified when creating KeyParams
         private const val POPUP_KEYS_NAVIGATE_PREVIOUS = "!icon/previous_key|!code/key_action_previous,!icon/clipboard_action_key|!code/key_clipboard"
         private const val POPUP_KEYS_NAVIGATE_NEXT = "!icon/clipboard_action_key|!code/key_clipboard,!icon/next_key|!code/key_action_next"
-        private const val POPUP_KEYS_NAVIGATE_PREVIOUS_NEXT = "!fixedColumnOrder!3,!needsDividers!,!icon/previous_key|!code/key_action_previous,!icon/clipboard_action_key|!code/key_clipboard,!icon/next_key|!code/key_action_next"
-        private const val POPUP_KEYS_NAVIGATE_EMOJI_PREVIOUS = "!fixedColumnOrder!3,!needsDividers!,!icon/previous_key|!code/key_action_previous,!icon/clipboard_action_key|!code/key_clipboard,!icon/emoji_action_key|!code/key_emoji"
+        private const val POPUP_KEYS_NAVIGATE_PREVIOUS_NEXT = "!fixedOrder!3,!needsDividers!,!icon/previous_key|!code/key_action_previous,!icon/clipboard_action_key|!code/key_clipboard,!icon/next_key|!code/key_action_next"
+        private const val POPUP_KEYS_NAVIGATE_EMOJI_PREVIOUS = "!fixedOrder!3,!needsDividers!,!icon/previous_key|!code/key_action_previous,!icon/clipboard_action_key|!code/key_clipboard,!icon/emoji_action_key|!code/key_emoji"
         private const val POPUP_KEYS_NAVIGATE_EMOJI = "!icon/clipboard_action_key|!code/key_clipboard,!icon/emoji_action_key|!code/key_emoji"
-        private const val POPUP_KEYS_NAVIGATE_EMOJI_NEXT = "!fixedColumnOrder!3,!needsDividers!,!icon/clipboard_action_key|!code/key_clipboard,!icon/emoji_action_key|!code/key_emoji,!icon/next_key|!code/key_action_next"
-        private const val POPUP_KEYS_NAVIGATE_EMOJI_PREVIOUS_NEXT = "!fixedColumnOrder!4,!needsDividers!,!icon/previous_key|!code/key_action_previous,!icon/clipboard_action_key|!code/key_clipboard,!icon/emoji_action_key|!code/key_emoji,!icon/next_key|!code/key_action_next"
+        private const val POPUP_KEYS_NAVIGATE_EMOJI_NEXT = "!fixedOrder!3,!needsDividers!,!icon/clipboard_action_key|!code/key_clipboard,!icon/emoji_action_key|!code/key_emoji,!icon/next_key|!code/key_action_next"
+        private const val POPUP_KEYS_NAVIGATE_EMOJI_PREVIOUS_NEXT = "!fixedOrder!4,!needsDividers!,!icon/previous_key|!code/key_action_previous,!icon/clipboard_action_key|!code/key_clipboard,!icon/emoji_action_key|!code/key_emoji,!icon/next_key|!code/key_action_next"
     }
 
     /** get the label, but also considers code, which can't be set separately for popup keys and thus goes into the label */
diff --git a/app/src/main/java/helium314/keyboard/latin/utils/PopupKeysUtils.kt b/app/src/main/java/helium314/keyboard/latin/utils/PopupKeysUtils.kt
@@ -38,17 +38,17 @@ fun createPopupKeysArray(popupSet: PopupSet<*>?, params: KeyboardParams, label:
     }
     if (!popupKeysDelegate.isInitialized() || popupKeys.isEmpty())
         return null
-    val fco = popupKeys.firstOrNull { it.startsWith(Key.POPUP_KEYS_FIXED_COLUMN_ORDER) }
-    if (fco != null && fco.substringAfter(Key.POPUP_KEYS_FIXED_COLUMN_ORDER).toIntOrNull() != popupKeys.size - 1) {
+    val fco = popupKeys.firstOrNull { it.startsWith(Key.POPUP_KEYS_FIXED_ORDER) }
+    if (fco != null && fco.substringAfter(Key.POPUP_KEYS_FIXED_ORDER).toIntOrNull() != popupKeys.size - 1) {
         val fcoExpected = popupKeys.size - popupKeys.count { it.startsWith("!") && it.endsWith("!") } - 1
-        if (fco.substringAfter(Key.POPUP_KEYS_FIXED_COLUMN_ORDER).toIntOrNull() != fcoExpected)
+        if (fco.substringAfter(Key.POPUP_KEYS_FIXED_ORDER).toIntOrNull() != fcoExpected)
             popupKeys.remove(fco) // maybe rather adjust the number instead of remove?
     }
     if (popupKeys.size > 1 && (label == "(" || label == ")")) { // add fixed column order for that case (typically other variants of brackets / parentheses
         // not really fast, but no other way to add first in a LinkedHashSet
         val tmp = popupKeys.toList()
         popupKeys.clear()
-        popupKeys.add("${Key.POPUP_KEYS_FIXED_COLUMN_ORDER}${tmp.size}")
+        popupKeys.add("${Key.POPUP_KEYS_FIXED_ORDER}${tmp.size}")
         popupKeys.addAll(tmp)
     }
     // autoColumnOrder should be fine
@@ -76,7 +76,7 @@ fun getHintLabel(popupSet: PopupSet<*>?, params: KeyboardParams, label: String):
         return null // better show nothing instead of the toolbar key label
 
     return KeySpecParser.getLabel(transformLabel(hintLabel, params))
-        // avoid e.g. !autoColumnOrder! as label
+        // avoid e.g. !autoOrder! as label
         //  this will avoid having labels on comma and period keys
         ?.takeIf { label -> !label.startsWith("!") || label.count { it == '!' } != 2 } // excluding the special labels
 }

Original file line number	Diff line number	Diff line change
`@@ -27,5 +27,5 @@ $$$4`
`27`	`27`	`[`
`28`	`28`	`]`
`29`	`29`
`30`		`-< !fixedColumnOrder!3 ‹ ≤ «`
`31`		`-> !fixedColumnOrder!3 › ≥ »`
	`30`	`+< !fixedOrder!3 ‹ ≤ «`
	`31`	`+> !fixedOrder!3 › ≥ »`
Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@`
`93`	`93`	`"popup": {`
`94`	`94`	`"relevant": [`
`95`	`95`	`{`
`96`		`- "label": "!fixedColumnOrder!2"`
	`96`	`+ "label": "!fixedOrder!2"`
`97`	`97`	`},`
`98`	`98`	`{`
`99`	`99`	`"label": "!hasLabels!"`
`@@ -115,7 +115,7 @@`
`115`	`115`	`"popup": {`
`116`	`116`	`"relevant": [`
`117`	`117`	`{`
`118`		`- "label": "!fixedColumnOrder!2"`
	`118`	`+ "label": "!fixedOrder!2"`
`119`	`119`	`},`
`120`	`120`	`{`
`121`	`121`	`"label": "!hasLabels!"`