Opened 3 years ago

Last modified 2 years ago

#6234 new defect

Hotkey bug to toggle window size while in lobby chat input

Reported by: Langbart Owned by:
Priority: Must Have Milestone: Backlog
Component: UI – In-game Keywords: input focus hotkeys disabled
Cc: Patch:

Description (last modified by Langbart)

Nani has noticed an error when the cursor is in the lobby chat input field and you try to toggle the window size via Alt+Enter. It does not work. The hotkey worked for A24.

I am creating the ticket so that the error is not forgotten.

related ticket

Attachments (1)

lobby_hotkey.jpg (30.9 KB ) - added by Langbart 3 years ago.

Download all attachments as: .zip

Change History (6)

by Langbart, 3 years ago

Attachment: lobby_hotkey.jpg added

comment:1 by Freagarach, 3 years ago

I am not sure this is a bug. It seems logical that having text input selected it catches *all* keys? That said, it sounds like something that would be nice indeed.

comment:2 by wraitii, 3 years ago

Priority: Should HaveNice to Have

This is sorta working as designed, see Phab:rP25180 . On a mac system (and possibly linux/windows as well, need some checking), alt can be used to input text, so shouldn't trigger hotkeys.

Alt+enter in particular is different from regular enter on regular MacOS keyboards, and so shouldn't trigger hotkeys in multiline input.

That said, the logic could possibly be improved so that if Enter isn't a possible input (e.g. not multiline stuff) then the hotkey could still be triggered, but that isn't something I'd call very worthwhile.

Keeping the ticket alive, but fix maybe unlikely.

comment:3 by nani, 3 years ago

Keywords: input focus hotkeys disabled added
Priority: Nice to HaveMust Have

Now you can't unfocus from the text input with any hotkey or key from the keyboard, now you must grab the mouse  like a caveman and click somewhere else on the screen to be able to use the usual GUI hotkeys. Note: pressing "Escape" seems to close properly the in-game chat only because it has overwritten it in the JS explicitly. Being able to at least have the unfocus event working should be better default (so this should be a must have)

comment:4 by Freagarach, 2 years ago

Milestone: Alpha 26Backlog

comment:5 by Langbart, 2 years ago

Description: modified (diff)

linking related ticket

Note: See TracTickets for help on using tickets.