sway: i3keys web - Supporting symbol key names starting with a capital letter.

[junaruga]

Hi, thanks for providing and maintaining this nice tool!

My environment

I am using sway on Fedora Linux 38.

$ sway --version
sway version 1.8.1

I am using the i3keys on the current latest master branch 99e368e.

The issue summary

In sway, it's possible to define symbol keys as the name starting with a capital letter first such as Return, Space, Semicolon, Comma, Period, Slash. I can see these keys are not shown as occupied (red color) on the page by the i3keys web.

I don't see the capital letter first names worked in i3 config except "Return". But it's nice if i3keys considers the capital letter first key names.

I am using this sway config.
https://github.com/junaruga/framework-laptop-config/blob/main/home/.config/sway/config

set $mod Mod4
...
bindsym $mod+Return exec $term
...
bindsym $mod+Shift+Semicolon exec $term
...
# KEY_DELETE 111
bindsym $mod+Comma exec "sudo ydotool key 111:1 111:0"
# KEY_INSERT 110
bindsym $mod+Period exec "sudo ydotool key 110:1 110:0"
# KEY_ENTER 28
bindsym $mod+Slash exec "sudo ydotool key 28:1 28:0"
...
bindsym $mod+Semicolon mode "symbol"
bindsym $mod+Shift+Space mode "symbol"

I was able to see the following $mod+minus was shown as occupied.

bindsym $mod+minus scratchpad show

Here is the actual screenshot. You can see these capital letter first keys are not shown as occupied (red).

[elder-n00b]

I confirm. Quick fix is normalizing case in

i3keys/keyboard/layouts.go

Line 101 in 99e368e

if symbol == key.Key {

if strings.ToLower(symbol) == strings.ToLower(key.Key)

There may be a better way but this seems to work.

On a side note, key symbols are always capitalized in

i3keys/helpers/parse.go

Line 52 in 99e368e

p := strings.Title(item)

but /usr/include/X11/keysymdef.h, which is parsed to generate

i3keys/xlib/keysyms.go

Line 4 in 99e368e

var KeySyms = map[string]string{

uses inconsistent capitalization.
That should explain why XK_space has the problem but XK_Return does not.

[junaruga]

I confirm. Quick fix is normalizing case in

i3keys/keyboard/layouts.go

Line 101 in 99e368e

if symbol == key.Key {

if strings.ToLower(symbol) == strings.ToLower(key.Key)

There may be a better way but this seems to work.

Thank you for taking a look at this issue, and suggesting the fix!

On a side note, key symbols are always capitalized in

i3keys/helpers/parse.go

Line 52 in 99e368e

p := strings.Title(item)

but /usr/include/X11/keysymdef.h, which is parsed to generate

i3keys/xlib/keysyms.go

Line 4 in 99e368e

var KeySyms = map[string]string{

uses inconsistent capitalization.
That should explain why XK_space has the problem but XK_Return does not.

Oh, I see. Thank you for providing this info. Now I understood why there are inconsistent symbol names in i3 config. It had been a mystery in my mind for a long term.

[junaruga]

I sent the PR #40!