Skip to content

Comments

Ignore ALT_GR when considering to hide content assists#3330

Open
fhackenberger wants to merge 1 commit intoeclipse-platform:masterfrom
fhackenberger:contentassist-altgr-nohide
Open

Ignore ALT_GR when considering to hide content assists#3330
fhackenberger wants to merge 1 commit intoeclipse-platform:masterfrom
fhackenberger:contentassist-altgr-nohide

Conversation

@fhackenberger
Copy link

@fhackenberger fhackenberger commented Sep 26, 2025

On Linux ALT_GR is the level3 switch (at least on the US-INTL layout), when using 'include "level3(YOURKEY)"' in the xkb file. Pressing it should leave the content assist open.

I've remapped Caps-Lock as the level3 switch and changed the keyboard layout, so I can use CapsLock-h/j/k/l instead of the arrow keys. This works in all applications I use on my Gnome desktop, except for the eclipse autocomplete popups, which is quite annoying as I'm using eclipse as my main IDE for backend development.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 27, 2025

Test Results

 3 024 files  ±0   3 024 suites  ±0   2h 14m 4s ⏱️ - 2m 40s
 8 234 tests ±0   7 984 ✅  - 2  248 💤 ±0  2 ❌ +2 
23 526 runs  ±0  22 733 ✅  - 2  791 💤 ±0  2 ❌ +2 

For more details on these failures, see this check.

Results for commit a4597e0. ± Comparison against base commit 89a19bf.

♻️ This comment has been updated with latest results.

@fhackenberger
Copy link
Author

The failed test NestedResourcesTest#testProjectHierarchy seems to be completely unrelated to my change.

@akurtakov akurtakov force-pushed the contentassist-altgr-nohide branch from f745bd6 to e82918e Compare October 3, 2025 18:32
@fhackenberger
Copy link
Author

@akurtakov I was wondering if your push means that you're looking into this to merge it? I'm not familiar with the workflow you're using for eclipse.

@akurtakov
Copy link
Member

In general yes - I'm just overloaded with things which leads to huge delays. Would you please rebase on master to fix conflicts?

@fhackenberger fhackenberger force-pushed the contentassist-altgr-nohide branch from e82918e to 623f591 Compare February 24, 2026 05:39
@fhackenberger
Copy link
Author

@akurtakov Sorry for the delay, I've rebased on master of today. Please see if you could merge it now.

On Linux ALT_GR is the level3 switch (at least on the US-INTL
layout), when using 'include "level3(YOURKEY)"' in the xkb file.
@fhackenberger fhackenberger force-pushed the contentassist-altgr-nohide branch from 623f591 to a4597e0 Compare February 24, 2026 05:42
@fhackenberger
Copy link
Author

fhackenberger commented Feb 24, 2026

Sorry I missed one of the conflicts. Fixed now.

Copy link
Member

@akurtakov akurtakov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now. We are in RC2 now so this will have to wait for next stream to be opened.

@BeckerWdf
Copy link
Member

Could that change have negative side effects on other platforms like Windows or macOS?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants