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

Bug 355413

Summary: Auto activation delay problems in EGL>Preferences>Content Assist
Product: z_Archived Reporter: broy2
Component: EDTAssignee: Xiao Bin Chen <xiaobinc>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: chenzhh
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Before
none
After
none
Patch for ca preference page. lasher: iplog+

Description broy2 CLA 2011-08-22 12:38:58 EDT
Build Identifier: 20110819

The Auto activation delay field has a couple of problems.
1.  The default is 500, so the field has room for 3 digits. But if I change the delay to 3 and OK to exit, the next time I open the preferences, the field has room for only 3 digit. It looks like it will accept more than 1 digit, but only 1 is displayed.
2.  When the field is blanked out, OK is disabled and the message displays:  positive integer must be specified.  But if I type in a 0, OK is enabled and the message goes away.  0 should behave like a blank.

Reproducible: Always

Steps to Reproduce:
1.EGL>Preferences>Content Assist
2.Change Auto activation delay to 3 and OK to exit.
3.Open preferences again.
4.The Auto activation delay field displays only 1 digit. 
5.Blank out the field and get error. Enter a 0 and the error goes away.
Comment 1 broy2 CLA 2011-08-22 12:40:54 EDT
Created attachment 201937 [details]
Before
Comment 2 broy2 CLA 2011-08-22 12:41:15 EDT
Created attachment 201938 [details]
After
Comment 3 broy2 CLA 2011-08-22 12:41:39 EDT
Screen shots of before and after attached.
Comment 4 Xiao Bin Chen CLA 2011-08-22 22:25:30 EDT
Hi tony.
The patch was provided. Please take a look and apply it.
Comment 5 Xiao Bin Chen CLA 2011-08-22 22:27:34 EDT
Created attachment 201964 [details]
Patch for ca preference page.

The patch 
1>disabled the auto-size of the textbox.
2>consider 0 condition.
Comment 6 Tony Chen CLA 2011-08-22 22:35:01 EDT
code committed
Comment 7 broy2 CLA 2011-08-23 16:56:58 EDT
20110823
Comment 8 broy2 CLA 2011-08-23 16:57:12 EDT
Verified