Bug 302171 - Characters requiring <Alt Gr> cannot be generated using Display.post(Event)
Summary: Characters requiring <Alt Gr> cannot be generated using Display.post(Event)
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 3.5.1   Edit
Hardware: PC Linux-GTK
: P3 normal (vote)
Target Milestone: 4.9 M2   Edit
Assignee: Xi Yan CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
: 280824 (view as bug list)
Depends on:
Blocks:
 
Reported: 2010-02-08 12:33 EST by Martin Hejl CLA
Modified: 2018-08-03 09:32 EDT (History)
10 users (show)

See Also:

Attachments
snippet (2.55 KB, application/octet-stream)
2018-05-16 03:18 EDT, Michael Seele CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Hejl CLA 2010-02-08 12:33:59 EST 
Characters that require the <Alt Gr> key (like Euro symbol or @ when using a German Keyboard) cannot be generated/sent using the Display.post() method.
On Windows, this can be worked around by using <CTRL>+<ALT> instead of <Alt Gr> - but on Linux (GTK) this has no effect.
It appears some SWT constant mapping to ISO_Level3_Shift (0xfe03) would be required in order to be able to send such characters using Display.post(Event)
Comment 1 Eric Williams CLA 2018-05-15 15:13:07 EDT 
Is this still an issue? Do you have a snippet which reproduces the issue?
Comment 2 Michael Seele CLA 2018-05-16 03:18:11 EDT 
Created attachment 274055 [details]
snippet
Comment 3 Michael Seele CLA 2018-05-16 03:18:27 EDT 
Yes, the issue still exists.
Basically it comes down to the fact that there is no SWT.ALT_GR constant we could use. 
Under windows, we can work around this by using SWT.CTRL + SWT.ALT, but under Linux, this has no effect (see attached snippet - under Windows, it produces "Love the method. @", under Linux (Ubuntu 16.04.4 LTS), it produces "Love the method. ")
Comment 4 Eric Williams CLA 2018-05-16 10:25:46 EDT 
(In reply to Michael Seele from comment #3)
> Yes, the issue still exists.
> Basically it comes down to the fact that there is no SWT.ALT_GR constant we
> could use. 
> Under windows, we can work around this by using SWT.CTRL + SWT.ALT, but
> under Linux, this has no effect (see attached snippet - under Windows, it
> produces "Love the method. @", under Linux (Ubuntu 16.04.4 LTS), it produces
> "Love the method. ")

I can reproduce this behaviour, thanks for the snippet.
Comment 5 Eclipse Genie CLA 2018-06-21 17:02:34 EDT 
New Gerrit change created: https://git.eclipse.org/r/124862
Comment 7 Xi Yan CLA 2018-06-26 15:21:04 EDT 
*** Bug 280824 has been marked as a duplicate of this bug. ***
Comment 8 Eclipse Genie CLA 2018-06-28 07:14:49 EDT 
New Gerrit change created: https://git.eclipse.org/r/125160
Comment 9 Andrey Loskutov CLA 2018-06-28 07:16:51 EDT 
(In reply to Eclipse Genie from comment #8)
> New Gerrit change created: https://git.eclipse.org/r/125160

This fixes the new API error, see bug 536384 comment 7.

@Xi: please install API tooling in your IDE and set baseline to 4.8.
Comment 11 Eric Williams CLA 2018-06-28 10:39:07 EDT 
Thanks for the API tools fix Andrey.

Adding Lakshmi and Niraj to CC: we have a new constant (ALT_GR) for German (possibly other European) keyboard layouts. Please implement (if possible) on Cocoa and Win32.
Comment 12 Eclipse Genie CLA 2018-06-29 15:51:19 EDT 
New Gerrit change created: https://git.eclipse.org/r/125273
Comment 14 Lakshmi P Shanmugam CLA 2018-07-12 07:37:00 EDT 
(In reply to Eric Williams from comment #11)
> Thanks for the API tools fix Andrey.
> 
> Adding Lakshmi and Niraj to CC: we have a new constant (ALT_GR) for German
> (possibly other European) keyboard layouts. Please implement (if possible)
> on Cocoa and Win32.

Opened Bug 536951 to track this and have set the helpwanted flag.
Comment 15 Lakshmi P Shanmugam CLA 2018-07-23 06:16:20 EDT 
(In reply to Eclipse Genie from comment #13)
> Gerrit change https://git.eclipse.org/r/125273 was merged to [master].
> Commit:
> http://git.eclipse.org/c/www.eclipse.org/eclipse/news.git/commit/?id=93012c1cb96882f2941c419af8d9a4b69d7100be
> 

Hi Eric, 
I see that the entry has been added to platform.html. New APIs are added to platform_isv.html. Can you please move the entry there?
Comment 16 Eclipse Genie CLA 2018-07-23 11:01:56 EDT 
New Gerrit change created: https://git.eclipse.org/r/126491