Skip to content

Ignore ALT_GR when considering to hide content assists - #3330

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

fhackenberger wants to merge 1 commit into
eclipse-platform:masterfrom
fhackenberger:contentassist-altgr-nohide

Conversation

@fhackenberger

@fhackenberger fhackenberger commented Sep 26, 2025 •

Copy link
Copy Markdown

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

github-actions Bot commented Sep 27, 2025 •

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

fhackenberger commented Feb 24, 2026 •

Copy link
Copy Markdown
Author

Sorry I missed one of the conflicts. Fixed now.

@akurtakov akurtakov left a comment

Copy link
Copy Markdown
Member

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
Copy Markdown
Member

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

@fhackenberger

Copy link
Copy Markdown
Author

@BeckerWdf I don't see how this could have negative side effects. ALT_GR is clearly a modifier key. Closing a pop-up with a modifier key doesn't really make sense on any platform, as they are often used for navigation.

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