Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Left Alt + space/shift expands the file menu but should do nothing. #3871

Closed
lampholder opened this issue May 10, 2017 · 12 comments
Closed

Left Alt + space/shift expands the file menu but should do nothing. #3871

lampholder opened this issue May 10, 2017 · 12 comments
Labels
A-Electron P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Z-Platform-Specific

Comments

@lampholder
Copy link
Member

Rageshake report from a user who uses left alt + space to change keyboard layout but Riot electron catches that and opens the file menu.

User reports that it is more normal for left alt press-and-release to open the file menu, and for left alt + space to be unbound and uncaptured.

This is likely platform specific (Linux) - I think Windows left-alt-plus-space might usually be bound.

@lampholder lampholder added Z-Platform-Specific A-Electron T-Defect S-Minor Impairs non-critical functionality or suitable workarounds exist P2 labels May 10, 2017
@t3chguy
Copy link
Member

t3chguy commented May 10, 2017

in Windows [l]Alt + Space opens the window management context menu, same as happens when you click the logo in top left of the window

@t3chguy
Copy link
Member

t3chguy commented May 10, 2017

Though I can not reproduce on my linux machine, [l]Alt + Space opens some search feature of KDE, so electron isn't catching it [l]Alt down and release shows the menu bar as expected.
Might need more information such as Distro and DE

@532910
Copy link
Contributor

532910 commented May 13, 2017

It was me.
It doesn't depend on WM, just on my xkb settings:

partial modifier_keys
xkb_symbols "alts-space-switch" {
	key <SPCE> {
		type = "MY_TYPE",
		symbols = [ space, ISO_First_Group, ISO_Last_Group ]
	};
};

@532910
Copy link
Contributor

532910 commented Jun 8, 2017

on my lalt-space xev shows:

KeyPress event, serial 39, synthetic NO, window 0x2a00001,
    root 0x4a8, subw 0x0, time 70841451, (93,37), root:(685,1278),
    state 0x0, keycode 64 (keysym 0xffe9, Alt_L), same_screen YES,
    XLookupString gives 0 bytes: 
    XmbLookupString gives 0 bytes: 
    XFilterEvent returns: False

KeyPress event, serial 39, synthetic NO, window 0x2a00001,
    root 0x4a8, subw 0x0, time 70841867, (93,37), root:(685,1278),
    state 0x8, keycode 65 (keysym 0xfe0c, ISO_First_Group), same_screen YES,
    XKeysymToKeycode returns keycode: 62
    XLookupString gives 0 bytes: 
    XmbLookupString gives 0 bytes: 
    XFilterEvent returns: False

KeyRelease event, serial 39, synthetic NO, window 0x2a00001,
    root 0x4a8, subw 0x0, time 70842091, (93,37), root:(685,1278),
    state 0x8, keycode 65 (keysym 0xfe0c, ISO_First_Group), same_screen YES,
    XKeysymToKeycode returns keycode: 62
    XLookupString gives 0 bytes: 
    XFilterEvent returns: False

KeyRelease event, serial 39, synthetic NO, window 0x2a00001,
    root 0x4a8, subw 0x0, time 70842459, (93,37), root:(685,1278),
    state 0x8, keycode 64 (keysym 0xffe9, Alt_L), same_screen YES,
    XLookupString gives 0 bytes: 
    XFilterEvent returns: False

@532910
Copy link
Contributor

532910 commented Jun 8, 2017

riot-web opens menu on the first "ISO_First_Group" in the sequence.

@t3chguy t3chguy changed the title Left Alt + space expands the file menu but should do nothing. Left Alt + space/shift expands the file menu but should do nothing. Jun 12, 2020
@VasoVV
Copy link

VasoVV commented Dec 17, 2020

Why reference to element-hq/element-desktop#787 is here?

The reporter (and me too) is not downthumb on the menu bar that appears when Alt is pressed and released.
That is normal behavior e.g. in Firefox. But Firefox doesn't open this menu when any key strokes with Alt are using, as Element does.
That is the bug that makes Element-desktop absolutely unusable for people who use these combinations.
For example I using Alt+Shift to switch keyboard layouts. During texting I could switch languages several times and every time I have to fight with file menu, that always appears after Alt is released.

Time to time I've install Element-desktop, and remove it after 5 minutes of endless fighting with menu bar.

The "severity" of this bug rather blocking than minor.

Additional observation:
If pressing Alt+Shift in "pure" way (Press Alt than Shift, release Shift than Alt), the menu doesn't appear.
But usually nobody presses keys this way -- that is mechanically poorly action.
Alt is always released the last and toggle annoying menu bar in Element-desktop.

@t3chguy
Copy link
Member

t3chguy commented Dec 17, 2020

as Element does.

Do you have this issue with any other modern Electron apps? Element doesn't do have any explicit handling for Alt, its all boilerplate Electron.

@VasoVV
Copy link

VasoVV commented Dec 17, 2020

I don't use any. But I've install typora for testing and have got the same behavior.

After some searching I've found that it is really an electron issue
And seems it wouldn't be resolve in the nearest future.

@532910
Copy link
Contributor

532910 commented Dec 17, 2020

Is this menu needed? What is it needed for?

modern Electron apps

Jitsi has no menu.
Drawio is completely unusable: menu is always visible, but it expands the only pane just by pressing (not releasing) alt-space (ISO_First_Group in my case)

@t3chguy
Copy link
Member

t3chguy commented Dec 17, 2020

The menu is required to bind to accelerators for simple key bindings whilst the window is in focus by the main process without external dependencies.

@xvenge00
Copy link

The issue is resolved in electron by PR. The fix is included in the latest release v12.0.11. So the bump in the should be sufficient. The patch is available in PR. It works for me on the element-desktop version 1.7.30, but i don't have time to check the develop branch.

@t3chguy
Copy link
Member

t3chguy commented Jun 10, 2022

Closing in favour of element-hq/element-desktop#787

@t3chguy t3chguy closed this as not planned Won't fix, can't repro, duplicate, stale Jun 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Electron P2 S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Z-Platform-Specific
Projects
None yet
Development

No branches or pull requests

5 participants