s3rius/squeekboard
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

keyboard.rs: Add keycode to the keymap with BackSpace if there is no other

Browse Source 
BackSpace does not work with `tools/entry.py` (made with GTK3),
if the keymap with BackSpace does not contain any other keycodes.
This should only happen for the emoji-layout or incomplete custom-layouts,
because the layout-tests for normal layouts
check for the presence of a button for Return.
This does add an "Unknown"-keycode, if necessary,
to let BackSpace work anyway.

Part-of: <https://gitlab.gnome.org/World/Phosh/squeekboard/-/merge_requests/695>
This commit is contained in:
MoonlightWave-12
2024-11-04 02:12:57 +01:00
parent 9fd15f76c7
commit 1a899f6b77
1 changed files with 14 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/keyboard.rs
View File

@ -117,7 +117,7 @@ pub fn generate_keycodes<'a, C: IntoIterator<Item=String>>(
let keycode_offset = 8;
HashMap::from_iter(
let mut keycode_map = HashMap::from_iter(
// Sort to remove a source of indeterminism in keycode assignment.
sorted(key_names.into_iter())
.zip(util::cycle_count((9..255).filter(|x| allowed.contains(&(x - keycode_offset)))))
@ -255,7 +255,19 @@ pub fn generate_keycodes<'a, C: IntoIterator<Item=String>>(
String::from(name),
KeyCode { code, keymap_idx },
)})
)
);
// Workaround: BackSpace does not work with `tools/entry.py` (made with GTK3),
// if the keymap with BackSpace does not contain any other keycodes.
// This should only happen for the emoji-layout or incomplete custom-layouts,
// because the layout-tests for normal layouts check for the presence of a button for Return.
// This does add an "Unknown"-keycode, if necessary, to let BackSpace work anyway.
if !HashMap::contains_key(&keycode_map, &"Return".to_string()) {
HashMap::insert(&mut keycode_map,
"Unknown".to_string(),
KeyCode { code: KEY_UNKNOWN + keycode_offset, keymap_idx: 0 }
);
}
keycode_map
}
#[derive(Debug)]