Skip to content

Commit

Permalink
Remove unused editing keys from the settings
Browse files Browse the repository at this point in the history
These keys don't seem to have a purpose, which is confusing.
  • Loading branch information
Julow committed Sep 3, 2023
1 parent 2349868 commit 816269a
Show file tree
Hide file tree
Showing 20 changed files with 7 additions and 57 deletions.
3 changes: 3 additions & 0 deletions check_layout.output
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,9 @@ Layout redefines the bottom row but some important keys are missing, missing: sw
2 warnings
# res/xml/latn_qwerty_br.xml
0 warnings
# res/xml/latn_qwerty_cz.xml
Layout doesn't define some important keys, missing: f11_placeholder, f12_placeholder
1 warnings
# res/xml/latn_qwerty_es.xml
0 warnings
# res/xml/latn_qwerty_hu.xml
Expand Down
3 changes: 1 addition & 2 deletions check_layout.py
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,7 @@ def check_layout(layout):
"Layout doesn't define some important keys")
unexpected_keys(keys,
["copy", "paste", "cut", "selectAll", "shareText",
"pasteAsPlainText", "undo", "redo", "replaceText",
"textAssist", "autofill" ],
"pasteAsPlainText", "undo", "redo" ],
"Layout contains editing keys")
unexpected_keys(keys,
[ "f1", "f2", "f3", "f4", "f5", "f6", "f7", "f8", "f9",
Expand Down
3 changes: 0 additions & 3 deletions res/values-cs/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<string name="key_descr_pasteAsPlainText">Vložit jako prostý text</string>
<string name="key_descr_undo">Zpět</string>
<string name="key_descr_redo">Znovu</string>
<string name="key_descr_replaceText">Nahradit</string>
<string name="key_descr_textAssist">Asistovat</string>
<string name="key_descr_autofill">Předvyplnit</string>
<string name="key_descr_ª">Indikátor řadové číslovky</string>
<string name="key_descr_º">Indikátor řadové číslovky</string>
<string name="key_descr_superscript">Horní index</string>
Expand Down
3 changes: 0 additions & 3 deletions res/values-de/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<string name="key_descr_pasteAsPlainText">Unformatiert einfügen</string>
<string name="key_descr_undo">Rückgängig</string>
<string name="key_descr_redo">Wiederholen</string>
<string name="key_descr_replaceText">Ersetzen</string>
<string name="key_descr_textAssist">Assistieren</string>
<string name="key_descr_autofill">Automatische Vervollständigung</string>
<string name="key_descr_ª">Ordinalzeichen</string>
<string name="key_descr_º">Ordinalzeichen</string>
<string name="key_descr_superscript">Hochgestellt</string>
Expand Down
3 changes: 0 additions & 3 deletions res/values-es/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<!-- <string name="key_descr_pasteAsPlainText">Paste as plain text</string> -->
<!-- <string name="key_descr_undo">Undo</string> -->
<!-- <string name="key_descr_redo">Redo</string> -->
<!-- <string name="key_descr_replaceText">Replace</string> -->
<!-- <string name="key_descr_textAssist">Assist</string> -->
<!-- <string name="key_descr_autofill">Autofill</string> -->
<!-- <string name="key_descr_ª">Ordinal Indicator</string> -->
<!-- <string name="key_descr_º">Ordinal Indicator</string> -->
<!-- <string name="key_descr_superscript">Superscript</string> -->
Expand Down
3 changes: 0 additions & 3 deletions res/values-fa/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<!-- <string name="key_descr_pasteAsPlainText">Paste as plain text</string> -->
<!-- <string name="key_descr_undo">Undo</string> -->
<!-- <string name="key_descr_redo">Redo</string> -->
<!-- <string name="key_descr_replaceText">Replace</string> -->
<!-- <string name="key_descr_textAssist">Assist</string> -->
<!-- <string name="key_descr_autofill">Autofill</string> -->
<!-- <string name="key_descr_ª">Ordinal Indicator</string> -->
<!-- <string name="key_descr_º">Ordinal Indicator</string> -->
<!-- <string name="key_descr_superscript">Superscript</string> -->
Expand Down
3 changes: 0 additions & 3 deletions res/values-fr/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<!-- <string name="key_descr_pasteAsPlainText">Paste as plain text</string> -->
<!-- <string name="key_descr_undo">Undo</string> -->
<!-- <string name="key_descr_redo">Redo</string> -->
<!-- <string name="key_descr_replaceText">Replace</string> -->
<!-- <string name="key_descr_textAssist">Assist</string> -->
<!-- <string name="key_descr_autofill">Autofill</string> -->
<!-- <string name="key_descr_ª">Ordinal Indicator</string> -->
<!-- <string name="key_descr_º">Ordinal Indicator</string> -->
<!-- <string name="key_descr_superscript">Superscript</string> -->
Expand Down
3 changes: 0 additions & 3 deletions res/values-it/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<!-- <string name="key_descr_pasteAsPlainText">Paste as plain text</string> -->
<!-- <string name="key_descr_undo">Undo</string> -->
<!-- <string name="key_descr_redo">Redo</string> -->
<!-- <string name="key_descr_replaceText">Replace</string> -->
<!-- <string name="key_descr_textAssist">Assist</string> -->
<!-- <string name="key_descr_autofill">Autofill</string> -->
<!-- <string name="key_descr_ª">Ordinal Indicator</string> -->
<!-- <string name="key_descr_º">Ordinal Indicator</string> -->
<!-- <string name="key_descr_superscript">Superscript</string> -->
Expand Down
3 changes: 0 additions & 3 deletions res/values-ko/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<!-- <string name="key_descr_pasteAsPlainText">Paste as plain text</string> -->
<!-- <string name="key_descr_undo">Undo</string> -->
<!-- <string name="key_descr_redo">Redo</string> -->
<!-- <string name="key_descr_replaceText">Replace</string> -->
<!-- <string name="key_descr_textAssist">Assist</string> -->
<!-- <string name="key_descr_autofill">Autofill</string> -->
<!-- <string name="key_descr_ª">Ordinal Indicator</string> -->
<!-- <string name="key_descr_º">Ordinal Indicator</string> -->
<!-- <string name="key_descr_superscript">Superscript</string> -->
Expand Down
3 changes: 0 additions & 3 deletions res/values-lv/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -95,9 +95,6 @@
<string name="key_descr_pasteAsPlainText">Ielīmēt kā vienkāršu tekstu</string>
<string name="key_descr_undo">Atsaukt</string>
<string name="key_descr_redo">Atatsaukt</string>
<string name="key_descr_replaceText">Aizvietot</string>
<string name="key_descr_textAssist">Palīgs</string>
<string name="key_descr_autofill">Automātiskā aizpilde</string>
<string name="key_descr_ª">Kārtas rādītājs</string>
<string name="key_descr_º">Kārtas rādītājs</string>
<string name="key_descr_superscript">Augšraksts</string>
Expand Down
3 changes: 0 additions & 3 deletions res/values-pl/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<string name="key_descr_pasteAsPlainText">Wklej sam tekst</string>
<string name="key_descr_undo">Cofnij</string>
<string name="key_descr_redo">Ponów</string>
<string name="key_descr_replaceText">Zamień</string>
<string name="key_descr_textAssist">Asystent</string>
<string name="key_descr_autofill">Autouzupełnianie</string>
<string name="key_descr_ª">Wskaźnik porządkowy (żeński)</string>
<string name="key_descr_º">Wskaźnik porządkowy (męski)</string>
<string name="key_descr_superscript">Indeks górny</string>
Expand Down
3 changes: 0 additions & 3 deletions res/values-pt/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<string name="key_descr_pasteAsPlainText">Colar texto não formatado</string>
<string name="key_descr_undo">Desfazer</string>
<string name="key_descr_redo">Refazer</string>
<string name="key_descr_replaceText">Substituir</string>
<string name="key_descr_textAssist">Assistente</string>
<string name="key_descr_autofill">Autopreenchimento</string>
<string name="key_descr_ª">Indicador Ordinal</string>
<string name="key_descr_º">Indicador Ordinal</string>
<string name="key_descr_superscript">Sobrescrito</string>
Expand Down
3 changes: 0 additions & 3 deletions res/values-ro/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<!-- <string name="key_descr_pasteAsPlainText">Paste as plain text</string> -->
<!-- <string name="key_descr_undo">Undo</string> -->
<!-- <string name="key_descr_redo">Redo</string> -->
<!-- <string name="key_descr_replaceText">Replace</string> -->
<!-- <string name="key_descr_textAssist">Assist</string> -->
<!-- <string name="key_descr_autofill">Autofill</string> -->
<!-- <string name="key_descr_ª">Ordinal Indicator</string> -->
<!-- <string name="key_descr_º">Ordinal Indicator</string> -->
<!-- <string name="key_descr_superscript">Superscript</string> -->
Expand Down
3 changes: 0 additions & 3 deletions res/values-ru/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<!-- <string name="key_descr_pasteAsPlainText">Paste as plain text</string> -->
<!-- <string name="key_descr_undo">Undo</string> -->
<!-- <string name="key_descr_redo">Redo</string> -->
<!-- <string name="key_descr_replaceText">Replace</string> -->
<!-- <string name="key_descr_textAssist">Assist</string> -->
<!-- <string name="key_descr_autofill">Autofill</string> -->
<!-- <string name="key_descr_ª">Ordinal Indicator</string> -->
<!-- <string name="key_descr_º">Ordinal Indicator</string> -->
<!-- <string name="key_descr_superscript">Superscript</string> -->
Expand Down
3 changes: 0 additions & 3 deletions res/values-tr/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<!-- <string name="key_descr_pasteAsPlainText">Paste as plain text</string> -->
<!-- <string name="key_descr_undo">Undo</string> -->
<!-- <string name="key_descr_redo">Redo</string> -->
<!-- <string name="key_descr_replaceText">Replace</string> -->
<!-- <string name="key_descr_textAssist">Assist</string> -->
<!-- <string name="key_descr_autofill">Autofill</string> -->
<!-- <string name="key_descr_ª">Ordinal Indicator</string> -->
<!-- <string name="key_descr_º">Ordinal Indicator</string> -->
<!-- <string name="key_descr_superscript">Superscript</string> -->
Expand Down
3 changes: 0 additions & 3 deletions res/values-vi/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<!-- <string name="key_descr_pasteAsPlainText">Paste as plain text</string> -->
<!-- <string name="key_descr_undo">Undo</string> -->
<!-- <string name="key_descr_redo">Redo</string> -->
<!-- <string name="key_descr_replaceText">Replace</string> -->
<!-- <string name="key_descr_textAssist">Assist</string> -->
<!-- <string name="key_descr_autofill">Autofill</string> -->
<!-- <string name="key_descr_ª">Ordinal Indicator</string> -->
<!-- <string name="key_descr_º">Ordinal Indicator</string> -->
<!-- <string name="key_descr_superscript">Superscript</string> -->
Expand Down
3 changes: 0 additions & 3 deletions res/values-zh-rCN/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<!-- <string name="key_descr_pasteAsPlainText">Paste as plain text</string> -->
<!-- <string name="key_descr_undo">Undo</string> -->
<!-- <string name="key_descr_redo">Redo</string> -->
<!-- <string name="key_descr_replaceText">Replace</string> -->
<!-- <string name="key_descr_textAssist">Assist</string> -->
<!-- <string name="key_descr_autofill">Autofill</string> -->
<!-- <string name="key_descr_ª">Ordinal Indicator</string> -->
<!-- <string name="key_descr_º">Ordinal Indicator</string> -->
<!-- <string name="key_descr_superscript">Superscript</string> -->
Expand Down
3 changes: 0 additions & 3 deletions res/values/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,6 @@
<string name="key_descr_pasteAsPlainText">Paste as plain text</string>
<string name="key_descr_undo">Undo</string>
<string name="key_descr_redo">Redo</string>
<string name="key_descr_replaceText">Replace</string>
<string name="key_descr_textAssist">Assist</string>
<string name="key_descr_autofill">Autofill</string>
<string name="key_descr_ª">Ordinal Indicator</string>
<string name="key_descr_º">Ordinal Indicator</string>
<string name="key_descr_superscript">Superscript</string>
Expand Down
5 changes: 0 additions & 5 deletions srcs/juloo.keyboard2/ExtraKeysPreference.java
Original file line number Diff line number Diff line change
Expand Up @@ -56,9 +56,6 @@ public class ExtraKeysPreference extends PreferenceCategory
"pasteAsPlainText",
"undo",
"redo",
"replaceText",
"textAssist",
"autofill",
"superscript",
"subscript",
};
Expand Down Expand Up @@ -92,8 +89,6 @@ static String key_description(Resources res, String name)
case "pasteAsPlainText": id = R.string.key_descr_pasteAsPlainText; break;
case "undo": id = R.string.key_descr_undo; break;
case "redo": id = R.string.key_descr_redo; break;
case "textAssist": id = R.string.key_descr_textAssist; break;
case "autofill": id = R.string.key_descr_autofill; break;
case "ª": id = R.string.key_descr_ª; break;
case "º": id = R.string.key_descr_º; break;
case "superscript": id = R.string.key_descr_superscript; break;
Expand Down
5 changes: 3 additions & 2 deletions srcs/juloo.keyboard2/KeyValue.java
Original file line number Diff line number Diff line change
Expand Up @@ -455,11 +455,12 @@ public static KeyValue getKeyByName(String name)
case "pasteAsPlainText": return editingKey(0xE035, Editing.PASTE_PLAIN);
case "undo": return editingKey(0xE036, Editing.UNDO);
case "redo": return editingKey(0xE037, Editing.REDO);
case "cursor_left": return editingKey(0xE008, Editing.CURSOR_LEFT);
case "cursor_right": return editingKey(0xE006, Editing.CURSOR_RIGHT);
// These keys are not used
case "replaceText": return editingKey("repl", Editing.REPLACE);
case "textAssist": return editingKey(0xE038, Editing.ASSIST);
case "autofill": return editingKey("auto", Editing.AUTOFILL);
case "cursor_left": return editingKey(0xE008, Editing.CURSOR_LEFT);
case "cursor_right": return editingKey(0xE006, Editing.CURSOR_RIGHT);

/* Placeholder keys */
case "removed": return placeholderKey(Placeholder.REMOVED);
Expand Down

0 comments on commit 816269a

Please sign in to comment.