Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 312215 - [Text] Text widget with default message needs a second click (IE)
Summary: [Text] Text widget with default message needs a second click (IE)
Status: RESOLVED FIXED
Alias: None
Product: RAP
Classification: RT
Component: RWT (show other bugs)
Version: 1.3   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 1.3 RC1   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-10 04:14 EDT by Niels Lippke CLA
Modified: 2010-05-12 09:21 EDT (History)
0 users

See Also:

Attachments
Proposed patch (714 bytes, patch)
2010-05-12 08:35 EDT, Ivan Furnadjiev CLA 		no flags Details | Diff
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 Niels Lippke CLA 2010-05-10 04:14:39 EDT 
When using a text widget with a default message, the text field does not 
handle the focus correctly: The message disappears, but there's no cursor 
and you also cannot type in any text. You have to click a second time upon 
the text field to get the cursor.
This seems to be an IE (8) issue, since I it works on FF.

Sample snippet:
public void createPartControl(Composite parent) {
parent.setLayout(new RowLayout());
Text text = new Text(parent, SWT.BORDER);
text.setMessage("enter");
}
Comment 1 Ivan Furnadjiev CLA 2010-05-12 04:56:19 EDT 
IE8, Safari 4.05 and Opera 10.53 have this problem, Chrome 5.0.396 and FF 3.6.3 work fine.
Comment 2 Ivan Furnadjiev CLA 2010-05-12 08:35:06 EDT 
Created attachment 168129 [details]
Proposed patch

This patch provides a workaround for the problem. It initializes buffered selectionStart and selectionLenght with zero. Thus, the first time focusing is forced by selection and the selection is set to 0, 0. In addition, this makes all browsers to have the same value returned by TextFiled#getSelectionStart - 0, instead -1 in IE, null in FF.
Comment 3 Rüdiger Herrmann CLA 2010-05-12 08:45:48 EDT 
The patch looks good, please go ahead and commit it.
Comment 4 Ivan Furnadjiev CLA 2010-05-12 09:21:21 EDT 
Applied patch to CVS HEAD.