Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 346171

Summary: Composition ignores typed key after pending accent
Product: [Eclipse Project] Platform Reporter: Hugo Corbucci <nightao>
Component: SWTAssignee: Platform-SWT-Inbox <platform-swt-inbox>
Status: CLOSED WORKSFORME QA Contact:
Severity: normal    
Priority: P3 CC: eclipse.felipe
Version: 3.6   
Target Milestone: ---   
Hardware: Macintosh   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:
Attachments:
Description Flags
StyledText's behavior of the bug
none
TextEdit's behavior of the bug none

Description Hugo Corbucci CLA 2011-05-17 19:03:40 EDT
Build Identifier: 20100617-1415

StyledText doesn't follow common behavior regarding pending accents on OS X. Whenever there is a pending composition followed by a non-accentuable key, that key is ignored to confirm the accent. Regular behavior is to confirm the accent and then add the key.
We tried to track down the issue and realized the problem is probably outside of the Java code. It looks like interpretKeyEvents on NSResponder might not be called with the proper arguments (maybe on the first keystroke or the second, couldn't identify which) by keyDown method on Composite.

Easy to reproduce with US-International but happens with any keyboard configuration while typing accents.

Reproducible: Always

Steps to Reproduce:
0. Set your keyboard to US-International
1. Create a StyledText
2. Type "'"
3. Type "s"
4. Text should be "'s" but is "'"
Comment 1 Hugo Corbucci CLA 2011-05-17 19:05:12 EDT
Created attachment 195918 [details]
StyledText's behavior of the bug

KeyCstr shows what is being typed. At the end, text should be "Let's" but it's only "Let'".
Comment 2 Hugo Corbucci CLA 2011-05-17 19:06:01 EDT
Created attachment 195919 [details]
TextEdit's behavior of the bug

This is how TextEdit works. Text is "Let's" at the end.
Comment 3 Felipe Heidrich CLA 2011-05-18 12:54:27 EDT
Works for me with 3.7 m7 on Mac OSX 10.6.7

can you please try with a newer version of eclipse ?
Comment 4 Hugo Corbucci CLA 2011-05-18 13:10:16 EDT
Indeed. Works fine on 3.7M7.
Sorry about that.